A text glitch was discovered on the group admin page, where a non-admin member sees a placeholder text ([en.admin.groups.selector_placeholder]) instead of the actual text. ChrisBeach created a PR to fix this issue, assuming that customized text with an “admin.” prefix is not available to non-admin clients.
sam pointed out the oddity of having an admin client localization file but not a moderator client localization file, suggesting centralization or splitting into “staff” client/server locale files. eviltrout agreed that the admin locale should apply to moderators and proposed renaming it to “staff” for clarity.
The discussion revealed that the bug might be related to the path or the fact that the admin locale is not loaded for non-staff group owners. Mittineague provided an example of the issue on a local development environment.
sam reviewed the change again and confirmed that it is correct, explaining that the issue arises when a non-staff group owner needs to access the public group edit tab, which shares locale stuff with the staff locale file.
The issue was eventually fixed by tgxworld in a commit to the Discourse repository.
When logged in as admin, I see “enter username” on the group editor, but when logged in as a non-admin member, I see “[en.admin.groups.selector_placeholder]”:
One odd thing here @eviltrout is that we have admin client localization file but do not have moderator client localization file. Would make sense either to centralize so we have “staff” client/server locale files, or split so there is both admin and moderator stuff.
I’ve always thought of the admin locale as “admin section” so it should apply to moderators too. I’m happy to rename it to staff if that makes it more clear. Either way if you are on an /admin path it should be loaded.
OH I just reviewed this again and think the change is correct (thought the github diff makes it very hard to understand)
What is happening is that you have a non staff group owner and need to allow the public group edit tab to work. It was sharing locale stuff with the staff locale file so it clearly fails.
@tgxworld can you review this and perhaps clean it up.
What we want to make sure is that we don’t have duplicate strings in admin/non-admin and duplicate templates here.