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

Remove publisher metadata component #3190

Merged
merged 2 commits into from
Jun 7, 2024

Conversation

andysellick
Copy link
Contributor

@andysellick andysellick commented May 17, 2024

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

What / why

Remove the 'published metadata' component: https://govuk-government-frontend.herokuapp.com/component-guide/publisher_metadata

  • appears to not be in use
  • functionality can be reproduced using the metadata component from govuk_publishing_components if we need it

Visual changes

None. This is what the component looks like (for some reason styles don't work properly for it in the government-frontend component guide).

Screenshot 2024-05-17 at 14 20 15

@govuk-ci govuk-ci temporarily deployed to government-frontend-pr-3190 May 17, 2024 13:20 Inactive
@andysellick andysellick force-pushed the remove-publisher-metadata-component branch from 705a04c to db38e65 Compare May 17, 2024 13:22
@govuk-ci govuk-ci temporarily deployed to government-frontend-pr-3190 May 17, 2024 13:23 Inactive
@andysellick andysellick marked this pull request as ready for review May 17, 2024 13:28
@andysellick andysellick requested review from kevindew and maxgds May 22, 2024 09:57
@kevindew
Copy link
Member

Nice work @andysellick I think you also have phrases in the locale files to remove for this (I searched for publisher_metadata)

There is also this line to adjust:

A replacement for the [metadata component](https://govuk-static.herokuapp.com/component-guide/metadata) with only the format specific details. This component separates facet metadata from [publisher metadata](/component-guide/publisher_metadata).

I think other search matches to publisher_metadata were unrelated.

- appears to not be in use
- functionality can be reproduced using the metadata component from govuk_publishing_components
@andysellick andysellick force-pushed the remove-publisher-metadata-component branch from db38e65 to ccaff84 Compare June 4, 2024 16:00
@govuk-ci govuk-ci temporarily deployed to government-frontend-pr-3190 June 4, 2024 16:00 Inactive
- remove text for publisher metadata component
@andysellick andysellick force-pushed the remove-publisher-metadata-component branch from ccaff84 to 7680daa Compare June 4, 2024 16:08
@govuk-ci govuk-ci temporarily deployed to government-frontend-pr-3190 June 4, 2024 16:08 Inactive
@andysellick
Copy link
Contributor Author

Thanks @kevindew have updated 👍

Copy link
Member

@kevindew kevindew left a 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 👍

@andysellick andysellick merged commit baf101e into main Jun 7, 2024
11 checks passed
@andysellick andysellick deleted the remove-publisher-metadata-component branch June 7, 2024 08:13
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