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

Updates placeholder schema references #3620

Merged
merged 1 commit into from
Sep 19, 2023

Conversation

1pretz1
Copy link
Contributor

@1pretz1 1pretz1 commented Sep 18, 2023

The placeholder schema was removed in 1, causing CI to fail. Swaps over to using guide.

@1pretz1 1pretz1 changed the title Updates placeholder references Updates placeholder schema references Sep 18, 2023
@govuk-ci govuk-ci temporarily deployed to components-gem-pr-3620 September 18, 2023 16:15 Inactive
@1pretz1 1pretz1 marked this pull request as ready for review September 18, 2023 16:19
@1pretz1 1pretz1 requested a review from KludgeKML September 18, 2023 16:27
Copy link
Contributor

@leenagupte leenagupte left a comment

Choose a reason for hiding this comment

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

Thanks for fixing this 🏅

One non-blocking comment, but otherwise. 👍

@@ -325,7 +325,7 @@ def given_there_is_a_non_step_by_step_parent_page
end

def not_step_by_step_content
not_step_by_step_content = random_item("placeholder")
not_step_by_step_content = random_item("generic")
Copy link
Contributor

Choose a reason for hiding this comment

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

Not a blocker, but I feel like we might end up having a similar problem with "generic" at some point. The only example in publishing-api lists "whitehall-frontend" as the rendering app, which is no longer true of anything.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good point 👍 I've switched all the references to guide now

Copy link
Contributor

@KludgeKML KludgeKML left a comment

Choose a reason for hiding this comment

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

LGTM (+ agree with Leena's comment)

The placeholder schema was removed in [1]. Swaps over to using `guide`.

[1]: alphagov/publishing-api#2493
@1pretz1 1pretz1 force-pushed the update-schema-placeholder-reference branch from 8a3cf17 to 8ac2b6a Compare September 19, 2023 08:43
@govuk-ci govuk-ci temporarily deployed to components-gem-pr-3620 September 19, 2023 08:43 Inactive
@1pretz1 1pretz1 merged commit 59a8993 into main Sep 19, 2023
6 checks passed
@1pretz1 1pretz1 deleted the update-schema-placeholder-reference branch September 19, 2023 08:48
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.

4 participants