-
Notifications
You must be signed in to change notification settings - Fork 12
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
馃悰 Fix Incorrect Default Title in Page Source #1352
Conversation
src/components/head/head.js
Outdated
const titles = { | ||
'/latest-rules/': 'Latest Rules', | ||
'/user/': 'User Rules', | ||
'/orphaned/': 'Orphaned Rules', | ||
'/archived/': 'Archived Rules', | ||
'/profile/': 'Profile', | ||
}; | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would it be worth moving this to a config file?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done 馃槈
src/helpers/config/pageTitles.js
Outdated
const titles = { | ||
'/latest-rules/': 'Latest Rules', | ||
'/user/': 'User Rules', | ||
'/orphaned/': 'Orphaned Rules', | ||
'/archived/': 'Archived Rules', | ||
'/profile/': 'Profile', | ||
}; | ||
|
||
export default titles; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we move this to https://github.com/SSWConsulting/SSW.Rules/blob/main/site-config.js
Just so its all in one spot?
Relates to #1333
This PR fixes the issue where the default
<title>
in the page source was incorrect for some pages.Figure: Orphaned Rule
Figure: Homepage