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

PCH: Fix some untranslatable strings #2323

Merged
merged 3 commits into from
Mar 26, 2024
Merged

Conversation

acicovic
Copy link
Collaborator

Description

This PR fixes a couple of strings that couldn't be translated, so they are translatable now.

Motivation and context

All our strings should be translatable.

How has this been tested?

Untested.

@acicovic acicovic added this to the 3.14.3 milestone Mar 26, 2024
@acicovic acicovic self-assigned this Mar 26, 2024
@acicovic acicovic marked this pull request as ready for review March 26, 2024 08:16
@acicovic acicovic requested a review from a team as a code owner March 26, 2024 08:16
@acicovic
Copy link
Collaborator Author

@vaurdan, on second pass, these strings seem a bit incorrect and would need to be updated?

@acicovic
Copy link
Collaborator Author

How about updating the string to "Settings"?

@vaurdan
Copy link
Contributor

vaurdan commented Mar 26, 2024

Yeah, they are definitely incorrect. Those are some left-over strings from when I was experimenting with the component before the final implementation. Changing them to Settings is definitely more appropriate :)

acicovic and others added 2 commits March 26, 2024 14:49
Co-authored-by: Henrique Mouta <henrique@automattic.com>
@acicovic acicovic requested a review from vaurdan March 26, 2024 13:17
@acicovic acicovic merged commit 55252a9 into develop Mar 26, 2024
36 checks passed
@acicovic acicovic deleted the fix/untranslatable-strings branch March 26, 2024 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants