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

refactor(i18n): ds-742 component, test updates #428

Merged
merged 1 commit into from
Sep 9, 2024

Conversation

cdcabrera
Copy link
Member

@cdcabrera cdcabrera commented Sep 6, 2024

What's included

  • app, clean unused not-so-dynamic locale prop
  • i18n, remove min used helper, add test
  • index, remove strictMode, interferes in i18next loading
  • sessionProvider, removed since only used locale

How to test

Coverage and basic unit test check

  1. update the NPM packages with $ npm install
  2. $ npm test
    1. confirm tests come back as expected

Local run check

  1. update the NPM packages with $ npm install
  2. $ npm run start
  3. confirm the display renders locale strings across all views

Example

...

Updates issue/story

DISCOVERY-742

* app, clean unused not-so-dynamic locale prop
* i18n, remove min used helper, add test
* index, remove strictMode, interferes in i18next loading
* sessionProvider, removed since only used locale

Relates to JIRA: DISCOVERY-742
@cdcabrera cdcabrera marked this pull request as ready for review September 9, 2024 16:21
@cdcabrera cdcabrera merged commit 1b0ac5e into quipucords:main Sep 9, 2024
3 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants