Skip to content

Conversation

catalinailie
Copy link
Contributor

@catalinailie catalinailie commented Aug 7, 2025

On non-english worldwide organisation pages, the link that should send the user to the world location news page is not working. For example, see Česká republika on this embassy page, that links to /world/ceska-republika/news , but it should be /world/czechia/news.
This is because in government-frontend we are using the country name from world_location_names, and content data returns the native country name for translated pages, and not the English version, which is the one we need.

Because world_locations doesn't return only the English version for location name, this issue is affecting links on Woldwide Organisation, News Articles, Case Studies and Speech pages.

Trello card: https://trello.com/c/dXfX6R0o/3433-fix-the-broken-link-in-the-head-of-some-non-english-worldwide-organisation-pages-that-links-to-the-world-location-news-page

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

This application is owned by the Whitehall Experience team. Please let us know in #govuk-whitehall-experience-tech when you raise any PRs.

Follow these steps if you are doing a Rails upgrade.

@catalinailie catalinailie force-pushed the return-english-name-in-world-location-names branch 6 times, most recently from c5461fd to 89141b4 Compare August 8, 2025 07:54
@catalinailie catalinailie force-pushed the return-english-name-in-world-location-names branch from 89141b4 to d0977e9 Compare August 8, 2025 08:37
@catalinailie
Copy link
Contributor Author

Actually I'm not really sure this is the correct solution. world_location_names is expected to store the native version of location name. (see commit where it was added) And it seems that world_locations is expected to have the locations in English.
At the moment both of them store the location in the language that the page is currently being rendered in. Not sure why or when the behaviour of world_locations has changed.

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.

1 participant