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 placeholder logic #8251

Merged
merged 1 commit into from
Sep 14, 2023
Merged

Remove placeholder logic #8251

merged 1 commit into from
Sep 14, 2023

Conversation

brucebolt
Copy link
Member

We were previously sending some content items to Publishing API with a schema type prefixed placeholder_ to indicate that content item was still rendered by Whitehall.

This is no longer the case, so we can remove that logic.

The GenericEditionPresenter is used solely in tests, reverting this to use a generic placeholder schema as the logic will be removed from Publishing API in alphagov/publishing-api#2504.

This will need to be updated again when the placeholder schema is completely removed.

Trello card

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

Follow these steps if you are doing a Rails upgrade.

We were previously sending some content items to Publishing API with a
schema type prefixed `placeholder_` to indicate that content item was
still rendered by Whitehall.

This is no longer the case, so we can remove that logic.

The `GenericEditionPresenter` is used solely in tests, reverting this to
use a generic `placeholder` schema as the logic will be removed from
Publishing API in alphagov/publishing-api#2504.

This will need to be updated again when the `placeholder` schema is
completely removed.
@brucebolt brucebolt marked this pull request as ready for review September 14, 2023 11:28
Copy link
Contributor

@jkempster34 jkempster34 left a comment

Choose a reason for hiding this comment

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

You mention that this will need updating further once the placeholder content schema is removed, do you mean beyond this? #8252

@brucebolt
Copy link
Member Author

You mention that this will need updating further once the placeholder content schema is removed, do you mean beyond this? #8252

Nope. Other than removing GenericEditionPresenter (which feels out of our team's scope), I think your PR covers the change that's needed here.

@brucebolt brucebolt merged commit b617964 into main Sep 14, 2023
9 checks passed
@brucebolt brucebolt deleted the remove-placeholder-logic branch September 14, 2023 14:38
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