-
Notifications
You must be signed in to change notification settings - Fork 45
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’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Mission statement clarification, standards refs #895
Conversation
- also update some references to standards, policies and guidance
Looking good. Makes sense to also update the site banner (layouts/custom.erb) with this batch of changes. Ok to include? |
Yes, I'm confused why the index page has its own layout. I'll try de-duplicating... |
- original change was in 89b0be6
- all the custom layout does is add a note about applicability, which we want on this page too
Have de-duplicated, I can't see anything too obvious breaking from changing the layout of the index page, but someone should probably double check for me :) |
Quick blame defines me as the culprit but I couldn't tell you why :) |
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.
Looks good to me!
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.
LGTM
Update mission statement and clarify applicability outside GDS.
Also update some references to standards, policies and guidance.
Plus fix the Dockerfile cos there's always some yak shaving to be done.