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

Translate "and" connective in metadata component for some languages #4477

Conversation

unoduetre
Copy link
Contributor

@unoduetre unoduetre commented Dec 5, 2024

What

Fix the following issue:

The From section on documents is missing translations for the word and see https://www.gov.uk/government/publications/uk-candidate-for-the-international-court-of-justice-election-2026-professor-dapo-akande-election-brochure.fr

The original ticket is about French and Spanish, as the authors are not sure which languages use "and" in the same way and which don't. I've extended the number of languages to the languages I know use "and" in the same way, namely Polish and German.

Why

Trello ticket

Visual Changes

Before

Screenshot 2024-12-05 at 14-01-03 Candidat du Royaume-Uni aux élections 2026 de la Cour internationale de justice brochure électorale du professeur Dapo Akande - GOV UK

After

Screenshot 2024-12-05 at 14-00-37 Candidat du Royaume-Uni aux élections 2026 de la Cour internationale de justice brochure électorale du professeur Dapo Akande - GOV UK

@govuk-ci govuk-ci temporarily deployed to components-gem-pr-4477 December 5, 2024 13:25 Inactive
@@ -20,6 +21,4 @@
</a>
</div>
<span id="toggle-<%= toggle_id %>" class="gem-c-metadata__toggle-items js-hidden"><%= remaining.to_sentence.html_safe %></span>
<% else %>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is a small refactoring I've done.

@unoduetre unoduetre changed the title Translate "and" connective in metadata component Translate "and" connective in metadata component for some languages Dec 5, 2024
@govuk-ci govuk-ci temporarily deployed to components-gem-pr-4477 December 5, 2024 13:40 Inactive
@unoduetre unoduetre force-pushed the 3082-add-the-translation-for-and-in-the-metadata-component-for-french-and-spanish-m branch from 793f72a to 800ecda Compare December 5, 2024 13:44
@govuk-ci govuk-ci temporarily deployed to components-gem-pr-4477 December 5, 2024 13:44 Inactive
@unoduetre unoduetre force-pushed the 3082-add-the-translation-for-and-in-the-metadata-component-for-french-and-spanish-m branch from 800ecda to 4267e37 Compare December 5, 2024 13:54
@govuk-ci govuk-ci temporarily deployed to components-gem-pr-4477 December 5, 2024 13:54 Inactive
@unoduetre unoduetre marked this pull request as ready for review December 5, 2024 14:03
@unoduetre unoduetre force-pushed the 3082-add-the-translation-for-and-in-the-metadata-component-for-french-and-spanish-m branch from 4267e37 to 1bec528 Compare December 5, 2024 14:05
@govuk-ci govuk-ci temporarily deployed to components-gem-pr-4477 December 5, 2024 14:06 Inactive
@unoduetre unoduetre force-pushed the 3082-add-the-translation-for-and-in-the-metadata-component-for-french-and-spanish-m branch from 1bec528 to 10aeaea Compare December 5, 2024 14:08
@govuk-ci govuk-ci temporarily deployed to components-gem-pr-4477 December 5, 2024 14:09 Inactive
@unoduetre unoduetre force-pushed the 3082-add-the-translation-for-and-in-the-metadata-component-for-french-and-spanish-m branch from 10aeaea to 22daff1 Compare December 5, 2024 14:10
@govuk-ci govuk-ci temporarily deployed to components-gem-pr-4477 December 5, 2024 14:10 Inactive
Copy link
Contributor

@hannako hannako left a comment

Choose a reason for hiding this comment

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

LGTM

@unoduetre unoduetre force-pushed the 3082-add-the-translation-for-and-in-the-metadata-component-for-french-and-spanish-m branch from 22daff1 to 1a1441b Compare December 6, 2024 11:33
@govuk-ci govuk-ci temporarily deployed to components-gem-pr-4477 December 6, 2024 11:34 Inactive
@unoduetre unoduetre force-pushed the 3082-add-the-translation-for-and-in-the-metadata-component-for-french-and-spanish-m branch from 1a1441b to 5ca6b40 Compare December 11, 2024 14:33
@govuk-ci govuk-ci temporarily deployed to components-gem-pr-4477 December 11, 2024 14:34 Inactive
@unoduetre unoduetre merged commit e0eb623 into main Dec 11, 2024
12 checks passed
@unoduetre unoduetre deleted the 3082-add-the-translation-for-and-in-the-metadata-component-for-french-and-spanish-m branch December 11, 2024 14:40
unoduetre added a commit that referenced this pull request Dec 11, 2024
* Add search-with-autocomplete to stylesheets served by static ([PR #4495](#4495))
* Add /media/ path to GA4 download link tracking ([PR #4491](#4491))
* Translate "and" connective in metadata component for some languages ([PR #4477](#4477))
@unoduetre unoduetre mentioned this pull request Dec 11, 2024
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