-
Notifications
You must be signed in to change notification settings - Fork 17
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
Add rendering for fields of operation #2711
Conversation
de192b0
to
9f617cf
Compare
The rendering for this is moving out of whitehall
9f617cf
to
2a1f861
Compare
This moves some hard-coded text out of the presenter
2a1f861
to
efe5cab
Compare
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.
sweet, thanks!
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! Could you also update the README before merging, plz :)
Thanks @AgaDufrat for the review!
Done, thanks for the reminder! I've put the status as 'migrated' which isn't immediately strictly true right now but we'll do the work to switch over rendering app immediately after this 😄 |
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!
We are moving the rendering for this page (/government/fields-of-operation) out of whitehall.
Screenshots:
Trello
Follow these steps if you are doing a Rails upgrade.