-
Notifications
You must be signed in to change notification settings - Fork 34
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
powerplatform-vscode Loc Checkin for WO 66644 #210
powerplatform-vscode Loc Checkin for WO 66644 #210
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After adding the translations-import/vscode-powerplatform.*.xlf
files, you will also need to build the repo locally to generate the translated json files, and check those in.
See Contributing.md for general instructions on that, though the gist is
- create an
msazure
org PAT here with the Packaging -> Read scope - store that PAT in an environment variable
AZ_DevOps_Read_PAT
- run
npm install
- run
npm run ci
- check in the generated
*.json
translation files
@tehcrashxor, I added the json files after fixing missing translations. Please review. Thanks a lot! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@v-hifuku-msft , looks like checked in the i18n/**/*.i18n.json
files generated by your local build, but did not check in the package.nls.*.json
files that the build generates at the root of the repository. Please commit those as well.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested the branch, and am successfully getting the updated strings in every part of the UI in the languages I tested.
Localization completed in Work Order 66644