Skip to content

Common action states and variants #219

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

Merged
merged 2 commits into from
Mar 27, 2025
Merged

Common action states and variants #219

merged 2 commits into from
Mar 27, 2025

Conversation

azisislm
Copy link
Collaborator

I'm adding to the shared localization file a handful of common action states which are duplicated frequently in existing localization definitions. The goal will be to use these moving forward when needing a generic action state localization, such as "Cancel" or "Done"

@github-actions github-actions bot added the CLA signed Indicates that all contributors have signed label Mar 26, 2025
@azisislm
Copy link
Collaborator Author

azisislm commented Mar 27, 2025

I moved the new localization keys to the rool level of the shared.json file, rather than all nested with commonActionStates, per suggestion from the team

The only hesitation I had with this was that the rest of the shared root is already a bit cluttered with some items which seem like they should go in other localization files - some non-global content mixed in

Think it's ok to put the new keys at root level too? just want to make sure we all like the structure, since hopefully these will be used a lot @yagil

@azisislm azisislm requested a review from yagil March 27, 2025 16:16
@azisislm azisislm merged commit ec9b6e8 into dev Mar 27, 2025
1 check passed
@github-actions github-actions bot locked and limited conversation to collaborators Mar 27, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CLA signed Indicates that all contributors have signed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants