Skip to content
This repository was archived by the owner on Aug 21, 2025. It is now read-only.

[Tooling] Add initial set of "frozen" strings for uploading to GlotPress#106

Merged
iangmaia merged 2 commits intotrunkfrom
iangmaia/add-initial-frozen-strings
Aug 11, 2025
Merged

[Tooling] Add initial set of "frozen" strings for uploading to GlotPress#106
iangmaia merged 2 commits intotrunkfrom
iangmaia/add-initial-frozen-strings

Conversation

@iangmaia
Copy link
Contributor

This PR is related to AINFRA-1034
Follow-up to 189316-ghe-Automattic/wpcom

Description

This PR simply copies the latest base strings from homeUi/src/main/res/values and loginUi/src/main/res/values to the "frozen" strings paths under fastlane/resources/frozen-strings.
This process should be automated once we introduce a code freeze routine, which will regularly copy the latest state of the strings when a release is started.

These strings will then be picked up by a cron job and will be uploaded as the latest source strings to GlotPress.

Testing Steps

Make sure the added strings are reflected in GlotPress once this PR is merged.

@iangmaia iangmaia requested review from a team and AdamGrzybkowski August 11, 2025 10:26
@iangmaia iangmaia self-assigned this Aug 11, 2025
@AliSoftware
Copy link
Contributor

Make sure the added strings are reflected in GlotPress once this PR is merged.

iinm this will require you to redo the PR on wpcom that got reverted by the i18n team since (189316-ghe-Automattic/wpcom#issuecomment-1063794) after this PR lands and the strings exist in trunk before we can see those strings in GlotPress though 😉

<string name="login_profile_load_failure_cta">Try again</string>
<string name="login_profile_load_failure_alternative_cta">Try with another account</string>
<string name="login_authorization_denied_title">You need to log in to Gravatar.com</string>
</resources> No newline at end of file
Copy link
Contributor

Choose a reason for hiding this comment

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

nitpick and unrelated to this PR per se, but this "no newline at end of file" bothers me a bit, would be nice to add it not only to this file but also to the original file this frozen-strings is a copy of 🙃

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh yes, added to them both at e6f8c1b 😌

@iangmaia iangmaia enabled auto-merge August 11, 2025 10:56
@AdamGrzybkowski
Copy link
Contributor

@iangmaia Do we need to manually update the frozen strings whenever we change the original files?

@iangmaia iangmaia merged commit 9e9f8a5 into trunk Aug 11, 2025
9 of 10 checks passed
@iangmaia iangmaia deleted the iangmaia/add-initial-frozen-strings branch August 11, 2025 11:05
@iangmaia
Copy link
Contributor Author

@AdamGrzybkowski

@iangmaia Do we need to manually update the frozen strings whenever we change the original files?

That's the idea if you want to have the strings uploaded to GlotPress. The idea is that once we have a regular release process with code freeze, a release branch, etc, we will "freeze" the strings automatically with automation.

We could in theory upload the latest strings directly from the project without "freezing" them, but that isn't recommended when you have a code freeze process as you want to make sure the strings for the version being released won't change.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants