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

[DO NOT MERGE] Remove metadata component #3682

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

MartinJJones
Copy link
Contributor

@MartinJJones MartinJJones commented Jul 5, 2023

What

Remove the metadata app component, replacing it with the metadata component from the publishing components gem.

Why

  • Using the metadata component from the publishing components gem helps to ensure that the design is consistent across GOV.UK
  • Helps keep our codebase DRY, we only need to make changes in one place

Trello: https://trello.com/c/XZyNCfWb/56-identify-which-components-could-be-improved-rationalised-on-govuk

Visual Changes

Before

app-c-metadata

After

gem-c-metadata

@govuk-ci govuk-ci temporarily deployed to govuk-frontend-app-pr-3682 July 5, 2023 14:17 Inactive
Update the view templates for the `bank-holiday` and `when-do-the-clocks-change` to use the `metadata` component from the publishing components gem.
`last_updated` was only being used in the `metadata` app component which has been removed in favour of the `metadata` component from the publishing components gem.
@govuk-ci govuk-ci temporarily deployed to govuk-frontend-app-pr-3682 July 5, 2023 14:20 Inactive
@govuk-ci govuk-ci temporarily deployed to govuk-frontend-app-pr-3682 July 5, 2023 14:29 Inactive
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.

None yet

2 participants