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.

Sign up for GitHub

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

Jump to bottom

frontend/admin: set the user UI lang temporarily to the one of the admin #8101

Merged

williamstein merged 1 commit into master from fix-admin-impersonate-lang_temp

Jan 8, 2025

Contributor

haraldschilly commented Jan 7, 2025 •

edited

Loading

auth/impersonate:
- change redirect target to .../app, to open the application directly
- if lang_temp is a locale, append it to that target URL
admin panel:
- the locale to switch to is the one of the admin (to stay consistent, instead of hardcoding "en")

haraldschilly added the PR-work in progress label

haraldschilly force-pushed the fix-admin-impersonate-lang_temp branch from d4fcb1e to 9789964 Compare

January 7, 2025 13:51

haraldschilly marked this pull request as ready for review

January 7, 2025 13:51


          frontend/admin: set the user UI lang temporarily to the one of the admin

7f31812

haraldschilly force-pushed the fix-admin-impersonate-lang_temp branch from 9789964 to 7f31812 Compare

January 7, 2025 13:53

haraldschilly added PR-needs review and removed PR-work in progress labels

williamstein merged commit 9c52117 into master

3 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR-needs review