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

[no-Jira] Upgrade i18next-parser #1188

Merged
merged 2 commits into from
Nov 11, 2024
Merged

[no-Jira] Upgrade i18next-parser #1188

merged 2 commits into from
Nov 11, 2024

Conversation

canac
Copy link
Contributor

@canac canac commented Nov 11, 2024

Description

Upgrade i18next-parser to fix a vulnerability in vue-template-compiler. Also, run yarn extract to verify that the output is the same with the old version and the new version.

Testing

The Run yarn extract commit was made with the old version as a reference point. To verify that the the new version produces the same output, run git restore public/locales/en/translation.json --source=main to revert that file back to its contents on main. Then run yarn extract and verify that there are no uncommitted changes. Without the changes to the config file, v9 produces empty strings for the values of any new labels it finds.

Checklist:

  • I have given my PR a title with the format "MPDX-(JIRA#) (summary sentence max 80 chars)"
  • I have applied the appropriate labels. (Add the label "On Staging" to get the branch automatically merged into staging.)
  • I have requested a review from another person on the project

@canac canac requested a review from dr-bizz November 11, 2024 17:32
@canac canac self-assigned this Nov 11, 2024
Copy link
Contributor

github-actions bot commented Nov 11, 2024

Bundle sizes [mpdx-react]

Compared against 09e40fd

No significant changes found

Copy link
Contributor

@dr-bizz dr-bizz left a comment

Choose a reason for hiding this comment

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

When testing, I found 4 words which were lost in your PR. I confirmed that these words weren't in your translation.json and they were in the repo.

Annual
Appointment
Archive
Initiation

@canac
Copy link
Contributor Author

canac commented Nov 11, 2024

When testing, I found 4 words which were lost in your PR. I confirmed that these words weren't in your translation.json and they were in the repo.

@dr-bizz Can you give me filenames and line numbers? I don't see t('Annual') or any of the other three labels in the repo on this branch.

@canac canac requested a review from dr-bizz November 11, 2024 20:29
Copy link
Contributor

@dr-bizz dr-bizz left a comment

Choose a reason for hiding this comment

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

You are correct, main also took all 4 of those phrases out.

I was reviewing it quickly, and saw annual in a test file, so there is no instances of those words in components.

Fixes a vulnerability in vue-template-compiler.
* Replace useKeysAsDefaultValue with a defaultValue function.
  * Documentation: https://github.com/i18next/i18next-parser/blob/master/docs/migration.md#migrating-from-6x-to-7x
* Remove nsSeparator because it was never a valid config field.
@canac canac force-pushed the dependabot-i18next-parser branch from da4d33d to 6212376 Compare November 11, 2024 21:10
@canac canac added the Preview Environment Add this label to create an Amplify Preview label Nov 11, 2024
Copy link
Contributor

Preview branch generated at https://dependabot-i18next-parser.d3dytjb8adxkk5.amplifyapp.com

@canac canac merged commit 9036e71 into main Nov 11, 2024
20 checks passed
@canac canac deleted the dependabot-i18next-parser branch November 11, 2024 21:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Preview Environment Add this label to create an Amplify Preview
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants