Skip to content
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

Merged

Conversation

unoduetre
Copy link
Contributor

@unoduetre unoduetre commented Feb 12, 2025

⚠️ This repo is Continuously Deployed: make sure you follow the guidance ⚠️

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

@govuk-ci govuk-ci temporarily deployed to govuk-frontend-app-pr-4638 February 12, 2025 09:22 Inactive
@unoduetre unoduetre force-pushed the 3259-incorrect-dates-for-consultations-on-get-involved-page branch from 92cc6b9 to f62032c Compare February 12, 2025 09:25
@govuk-ci govuk-ci temporarily deployed to govuk-frontend-app-pr-4638 February 12, 2025 09:25 Inactive
@unoduetre unoduetre force-pushed the 3259-incorrect-dates-for-consultations-on-get-involved-page branch from f62032c to 69ea3cd Compare February 12, 2025 10:08
@govuk-ci govuk-ci temporarily deployed to govuk-frontend-app-pr-4638 February 12, 2025 10:08 Inactive
@unoduetre unoduetre force-pushed the 3259-incorrect-dates-for-consultations-on-get-involved-page branch from 69ea3cd to 00b9663 Compare February 12, 2025 10:28
@govuk-ci govuk-ci temporarily deployed to govuk-frontend-app-pr-4638 February 12, 2025 10:28 Inactive
@unoduetre unoduetre force-pushed the 3259-incorrect-dates-for-consultations-on-get-involved-page branch from 00b9663 to c97f884 Compare February 12, 2025 14:48
@govuk-ci govuk-ci temporarily deployed to govuk-frontend-app-pr-4638 February 12, 2025 14:48 Inactive
@unoduetre unoduetre force-pushed the 3259-incorrect-dates-for-consultations-on-get-involved-page branch from c97f884 to c7aa8e0 Compare February 17, 2025 10:46
@govuk-ci govuk-ci temporarily deployed to govuk-frontend-app-pr-4638 February 17, 2025 10:46 Inactive
@unoduetre unoduetre marked this pull request as ready for review February 17, 2025 10:50
Copy link
Contributor

@catalinailie catalinailie left a 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.

@@ -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
Copy link
Contributor

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.

Copy link
Contributor Author

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.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed.

@unoduetre unoduetre force-pushed the 3259-incorrect-dates-for-consultations-on-get-involved-page branch from c7aa8e0 to 783c92f Compare February 17, 2025 13:11
@govuk-ci govuk-ci temporarily deployed to govuk-frontend-app-pr-4638 February 17, 2025 13:11 Inactive
@unoduetre unoduetre merged commit fa3d76c into main Feb 17, 2025
13 checks passed
@unoduetre unoduetre deleted the 3259-incorrect-dates-for-consultations-on-get-involved-page branch February 17, 2025 14:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants