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

TRCL-3542 Add global command strings for translation #256

Merged
merged 4 commits into from
Feb 2, 2024
Merged

Conversation

moo-onthelawn
Copy link
Contributor

@moo-onthelawn moo-onthelawn commented Feb 2, 2024

Internationalize strings from useGlobalCommands.

There was also a duplicate key of "POWERED_BY" - removed the unused one (the autogenerate script only maps one of the instances of the key) and replaced the existing one to be fully capitalized. update: looks like it's used by v4-native-ios, so didn't want to delete

  • Note: I deleted the old translations of "POWERED_BY" in the other language files so they could be re-translated - is that the right way of going about things? Or should I keep them in?

Copy link

vercel bot commented Feb 2, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
v4-localization ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 2, 2024 9:16pm

@moo-onthelawn moo-onthelawn changed the title Add global command strings for translation TRCL-3542 Add global command strings for translation Feb 2, 2024
Copy link

linear bot commented Feb 2, 2024

@@ -1340,7 +1340,6 @@
"LAUNCH_INCENTIVES_DESCRIPTION":"Each season, you can earn rewards for trading on dYdX Chain. Trade to earn your spot on the leaderboard.",
"LEADERBOARD": "Leaderboard",
"POINTS":"{POINTS} points",
"POWERED_BY":"POWERED BY",
Copy link
Contributor Author

@moo-onthelawn moo-onthelawn Feb 2, 2024

Choose a reason for hiding this comment

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

duplicate key (it's already in app.general.powered_by so this isn't even recognized in the generated app.ts file) ((hence, this isn't used anywhere in v4-web))

@@ -186,7 +186,6 @@
"PERPETUALS": "Perpetuals",
"PNL": "G&V",
"POSITION": "Position",
"POWERED_BY": "Powered By",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

deleted non-en translations of POWERED_BY so that it could be retranslated in caps lock - not sure if this is the right protocol - should I keep them in instead?

Copy link
Contributor

Choose a reason for hiding this comment

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

it's likely that Lilt will just replace these back with the original "Powered By" -- I usually just rename the key, so you can say POWERED_BY_ALL_CAPS: "POWERED BY" and that guarantees they'll translate based on the newest value

@moo-onthelawn moo-onthelawn marked this pull request as ready for review February 2, 2024 17:00
Copy link
Contributor

@aforaleka aforaleka left a comment

Choose a reason for hiding this comment

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

also need to bump version! (scripts/bump_version.sh)

can leave the translated powered by as is, or just create a new string and update usage (ios+web)!

@moo-onthelawn
Copy link
Contributor Author

also need to bump version! (scripts/bump_version.sh)

ah ty - done!

can leave the translated powered by as is, or just create a new string and update usage (ios+web)!

ooh created a new string, will follow up to update usage

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants