-
Notifications
You must be signed in to change notification settings - Fork 265
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
Move console related i18n extensions to @wso2is/i18n
module
#7320
Move console related i18n extensions to @wso2is/i18n
module
#7320
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #7320 +/- ##
=======================================
Coverage 36.53% 36.53%
=======================================
Files 42 42
Lines 906 906
Branches 223 223
=======================================
Hits 331 331
+ Misses 575 529 -46
- Partials 0 46 +46
Flags with carried forward coverage won't be shown. Click here to find out more. |
2862e0b
to
67b921e
Compare
features/admin.application-roles.v1/components/application-roles-list.tsx
Show resolved
Hide resolved
Let's include the reason for this change in the PR description. This will serve as a reference for the refactoring in case any issues arise from moving it. |
…ations to `@wso2is/i18n`
…ionRoles namespace
67b921e
to
bc94c44
Compare
🦋 Changeset detectedThe changes in this PR will be included in the next version bump. Not sure what this means? Click here to learn what changesets are. |
Purpose
There are some i18n strings inside console/extensions folder. Since the console is now unified, we can move this i18n to i18n module to be used in both onprem and asgardeo.
This issue focuses on moving the i18n under "console" category[1] in console/extensions directory.
[1]
identity-apps/apps/console/src/extensions/i18n/resources/en-US/extensions.ts
Lines 29 to 376 in 4230aa5
Related Issues
Related PRs
Checklist
Security checks