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

Delete placeholder translations #2445

Merged
merged 2 commits into from
Jul 13, 2023
Merged

Conversation

jkempster34
Copy link
Contributor

@jkempster34 jkempster34 commented Jul 12, 2023

https://trello.com/c/xxdiO4lZ

Note
In order to test this I have:

Delete placeholder translations

These placeholder translations get automatically expanded into the
available_translations for their content item, but they are not valid for
display, so they cause errors when using government-frontend to render
worldwide organisations.

Fix method name in outdated documentation

This method was renamed in the past (#683), but the relevant documentation
still has the old method name.


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

This application is owned by the publishing platform team. Please let us know in #govuk-publishing-platform when you raise any PRs.

Follow these steps if you are doing a Rails upgrade.

@jkempster34 jkempster34 force-pushed the 732-delete-placeholder-translations branch from 5e5dd69 to 34134d1 Compare July 13, 2023 09:55
jkempster34 and others added 2 commits July 13, 2023 12:13
These placeholder translations get automatically expanded into the
`available_translations` for their content item, but they are not valid for
display, so they cause errors when using government-frontend to render
worldwide organisations.

Co-authored-by: Cristina <[email protected]>
This method was renamed in the past (#683), but the relevant documentation
still has the old method name.

Co-authored-by: Cristina <[email protected]>
@CristinaRO CristinaRO force-pushed the 732-delete-placeholder-translations branch from 34134d1 to f77d62a Compare July 13, 2023 11:13
@jkempster34 jkempster34 merged commit 897aa03 into main Jul 13, 2023
7 checks passed
@jkempster34 jkempster34 deleted the 732-delete-placeholder-translations branch July 13, 2023 11:57
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