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

Make the metadata block on specialist documents optional #3241

Merged
merged 2 commits into from
Apr 1, 2025

Conversation

GDSNewt
Copy link
Contributor

@GDSNewt GDSNewt commented Mar 28, 2025

These change is necessary to make the rendering the blue metadata box at the top of specialist documents dependent on a value defined in the schema of each finder.

Associated PRs

  1. Government-Frontend
  2. Specialist-Publisher

Trello

We need to add this property to make the metadata block configurable via finder schema.
A conditional on the frontend can check for this property to determine whether or not to
render the metadata blue box.
@GDSNewt GDSNewt force-pushed the toggle-metadata-block-on-specialist-documents branch from 030e961 to fb4c21f Compare March 28, 2025 09:52
@GDSNewt GDSNewt force-pushed the toggle-metadata-block-on-specialist-documents branch from fb4c21f to 36bfef3 Compare March 28, 2025 11:58
@GDSNewt GDSNewt force-pushed the toggle-metadata-block-on-specialist-documents branch from 36bfef3 to 73d930f Compare March 31, 2025 08:51
Copy link
Contributor

@ChrisBAshton ChrisBAshton left a comment

Choose a reason for hiding this comment

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

LGTM 👍

Some tests on government frontend use these examples for their rendering tests.

I've made drug device alerts and countryside stewardship grants the 'true' cases for rendering the
metadata block.

Cma cases not having the property is acting as the 'false', i.e do not show the block.

We have plans to remove remove all usages of example schema files for Specialist Documents only.
It should generate randomly from a schema file, or load from a fixture.
The goal is so we can delete the example schema files from Publishing API.

Trello backlog ticket(https://trello.com/c/7vhL24Fk/840-update-government-frontend-tests-to-use-randomexample-generation-from-schemas-so-we-can-delete-specialist-document-examples-from)
@GDSNewt GDSNewt force-pushed the toggle-metadata-block-on-specialist-documents branch from 73d930f to a3a84bd Compare April 1, 2025 08:08
@GDSNewt GDSNewt merged commit 68aed41 into main Apr 1, 2025
40 checks passed
@GDSNewt GDSNewt deleted the toggle-metadata-block-on-specialist-documents branch April 1, 2025 08:17
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.

2 participants