Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Fix: Added faces context as source of locale #1189

Merged
merged 1 commit into from
Apr 11, 2022
Merged

Fix: Added faces context as source of locale #1189

merged 1 commit into from
Apr 11, 2022

Conversation

Milton-Ch
Copy link
Contributor

@Milton-Ch Milton-Ch commented Apr 11, 2022 •
edited
Loading

Added FacesContext to get ui locale.

#362

@mo-auto mo-auto added the comp-jans-auth-server Component affected by issue or PR label Apr 11, 2022
@sonarqubecloud
Copy link

[Jans authentication server parent] Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell D 2 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@Milton-Ch Milton-Ch marked this pull request as ready for review April 11, 2022 14:21
@Milton-Ch Milton-Ch requested review from yurem and yuriyz as code owners April 11, 2022 14:21
@yuriyz yuriyz merged commit ce770ae into main Apr 11, 2022
@yuriyz yuriyz deleted the fix/language-bean-no-ui-locale branch April 11, 2022 14:28
@mo-auto mo-auto mentioned this pull request May 18, 2022
Merged
@mo-auto3 mo-auto3 mentioned this pull request May 18, 2022
Merged
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@yuriyz yuriyz yuriyz approved these changes

@yurem yurem Awaiting requested review from yurem yurem is a code owner

Assignees
No one assigned
Labels
comp-jans-auth-server Component affected by issue or PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Milton-Ch @yuriyz @mo-auto