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

Add temporary container to published-dates #2330

Merged
merged 1 commit into from
Jan 7, 2022

Conversation

danacotoran
Copy link
Contributor

@danacotoran danacotoran commented Jan 7, 2022

This is to prevent the change in this PR being a breaking one:
alphagov/govuk_publishing_components#2558

Once the change above has been rolled out to government-frontend, the
additional container can be removed, and the id of the component wrapper updated
from "history" to "full-publication-update-history"

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

This is to prevent the change in this PR being a breaking one:
alphagov/govuk_publishing_components#2558

Once the change above has been rolled out to government-frontend, the
additional container can be removed, and the id of the container updated
from "history" to "full-publication-update-history"
@danacotoran danacotoran force-pushed the temp-workaround-for-metadata-link branch from fc045f7 to e094ba2 Compare January 7, 2022 16:30
@govuk-ci govuk-ci temporarily deployed to government-f-temp-worka-uqqbkb January 7, 2022 16:31 Inactive
Copy link
Contributor

@owenatgov owenatgov left a comment

Choose a reason for hiding this comment

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

Sneaky

@danacotoran danacotoran merged commit ea9bb28 into main Jan 7, 2022
@danacotoran danacotoran deleted the temp-workaround-for-metadata-link branch January 7, 2022 16:50
danacotoran added a commit that referenced this pull request Jan 17, 2022
PR #2330 added an additional div around published-dates to prevent this
change in govuk_publishing_components being a breaking one:
alphagov/govuk_publishing_components#2558

This additional div is now safe to remove and the id of the wrapper for
published-dates can be updated to the new one.
danacotoran added a commit that referenced this pull request Jan 17, 2022
PR #2330 added an additional div around published-dates to prevent this
change in govuk_publishing_components being a breaking one:
alphagov/govuk_publishing_components#2558

This additional div is now safe to remove and the id of the wrapper for
published-dates can be updated to the new one.
danacotoran added a commit that referenced this pull request Feb 1, 2022
…ata-link

Undo temporary workaround in pull request #2330
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