-
Notifications
You must be signed in to change notification settings - Fork 20
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
FIx incorrect dates in 'Get involved' page #4638
FIx incorrect dates in 'Get involved' page #4638
Conversation
92cc6b9
to
f62032c
Compare
f62032c
to
69ea3cd
Compare
69ea3cd
to
00b9663
Compare
00b9663
to
c97f884
Compare
c97f884
to
c7aa8e0
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.
The changes look good. I have only one small change request.
spec/system/get_involved_spec.rb
Outdated
@@ -35,6 +35,14 @@ | |||
it "does not display a single page notification button" do | |||
expect(page).not_to have_css(".gem-c-single-page-notification-button") | |||
end | |||
|
|||
it "shows opening date" do |
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.
I think this needs to be something like "shows when it was last updated" or other name that aligns with what we check.
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.
Yes, I forgot to change that from the previous version. I'll fix it in a minute.
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.
Fixed.
c7aa8e0
to
783c92f
Compare
What
The get involved page on gov.uk includes a 'Engage with government' section which lists recently opened consultations and consultations with recent outcomes.
The dates in these 2 sections ('Recently opened' and 'Recent outcomes') next to the department acronym (not the 'Closes'/'Closed' date) doesn't seem to relate to the document at all - either when it was first published or last updated.
Why
Trello card
Testing
Before
https://www.gov.uk/government/get-involved
After
https://govuk-frontend-app-pr-4638.herokuapp.com/government/get-involved