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

fix: Respect translation overrides for errors.E0xxx in i18n of widget config #3744

Merged
merged 4 commits into from
Nov 5, 2024

Conversation

denysoblohin-okta
Copy link
Contributor

@denysoblohin-okta denysoblohin-okta commented Oct 28, 2024

Description:

Fixes an issue introduced in #3579

In that PR error message for code E0000068 stoped to being localized (because there is no localization for it in default translation files bundled with SIW) and started to be taken from errorSummary in Authn API response.
But customer should be allowed to add translation overrides for any error codes in i18n of widget config.

PR Checklist

Issue:

Reviewers:

Screenshot/Video:

loc-fix.mov

Downstream Monolith Build:

@denysoblohin-okta denysoblohin-okta changed the title fix: Allow custom translation for errors.E00xxx fix: Allow custom translation for errors.E0xxx Oct 28, 2024
@denysoblohin-okta denysoblohin-okta changed the title fix: Allow custom translation for errors.E0xxx fix: Respect translation overrides for errors.E0xxx in i18n of widget config Oct 28, 2024
@denysoblohin-okta denysoblohin-okta marked this pull request as ready for review October 28, 2024 13:16
@denysoblohin-okta denysoblohin-okta force-pushed the od-fix-error-loc-OKTA-819400 branch from d385977 to dce46e3 Compare November 5, 2024 16:57
@oktapp-aperture-okta oktapp-aperture-okta bot merged commit 49d7f7b into 7.25 Nov 5, 2024
1 check passed
@oktapp-aperture-okta oktapp-aperture-okta bot deleted the od-fix-error-loc-OKTA-819400 branch November 5, 2024 19:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants