- Issue created by @prudloff
- ๐ฎ๐ณIndia Indranil Roy kolkata
Indranil Roy โ made their first commit to this issueโs fork.
- Merge request !53363915: Added 2FA setting page to admin theme. โ (Closed) created by Indranil Roy
- ๐ฎ๐ณIndia Indranil Roy kolkata
Hey @prudloff, I have added the commit `3363915: Added 2FA setting page to admin theme.` and raised the MR please review.
- Status changed to Needs review
9 months ago 7:52am 9 February 2024 - ๐ต๐ฐPakistan Ahmed.Raza
I disagree, this settings page is accessible by the normal users as well which are not admins, setting '_admin_route: TRUE' forces admin theme for everyone including the non-admin users.
I think how it is now is perfect, admins can use the normal site theme its not the end of the world.
Thanks
- Status changed to Closed: works as designed
9 months ago 3:31pm 15 February 2024 - ๐ซ๐ทFrance prudloff Lille
Normal users don't have the "see admin theme" permission and will see the page with the front theme even if it has "_admin_route: TRUE".
This is how the user edit page works for example. - ๐ต๐ฐPakistan Ahmed.Raza
I tested your solution and it was taking a normal user to admin theme as well. I feel like there is no need for switching themes for anybody in any case in the first place.
- ๐ซ๐ทFrance prudloff Lille
Do your users have the
view the administration theme
permission?