search

dnnsoftware / Dnn.Platform

DNN (formerly DotNetNuke) is the leading open source web content management platform (CMS) in the Microsoft ecosystem.
https://dnncommunity.org/
MIT License
1.02k stars 748 forks source link

[Enhancement]: Replace "All users" with "Everyone" #6091

Open Timo-Breumelhof opened 3 months ago

Timo-Breumelhof commented 3 months ago

Is there an existing issue for this?

Description of problem

Please read: https://dnncommunity.org/forums/future-development/dnn-ideas/quotall-usersquot-is-not-the-best-term-for-security/

Description of solution

Use "Everyone" instead of "All Users" as text for that "virtual role"

Description of alternatives considered

All visitors, but most seem to prefer "Everyone", see Forum Thread.

Anything else?

No response

Do you be plan to contribute code for this enhancement?

Would you be interested in sponsoring this enhancement?

Code of Conduct

Timo-Breumelhof commented 3 months ago

Hmm, seems to be quite a change if I look at the code..

sleupold commented 3 months ago

You should chenge just the localization value of Key AllUsers.text in App_GlobalRessources/SharedResources.resx file.

Timo-Breumelhof commented 3 months ago

The Persona Bar ignores that localized string, does work in module settings

sleupold commented 3 months ago

Timo, you are correct - but this is a separate bug IMHO. It shoould use localized values as well.

Timo-Breumelhof commented 3 months ago

Agreed