Skip to content
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.

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

feat: User settings not set if not in globals. #7194

Closed
sjpadgett opened this issue Jan 26, 2024 · 0 comments · Fixed by #7195
Closed

feat: User settings not set if not in globals. #7194

sjpadgett opened this issue Jan 26, 2024 · 0 comments · Fixed by #7195
Assignees
Labels
developers This issue targets an issue that is for developers/collaborators/module writers/technical users Module Support Use if effects modules
Milestone

Comments

@sjpadgett
Copy link
Sponsor Member

Describe the problem

Currently user settings must match a setting value set in globals to be included in the $GLOBAL super. If a module wants to set a needed setting that is user specific and does not want a default from global setting then the setting is never set in globals.php. E.g. Module wants have a global admin value for one reason but doesn't want that value to be valid for a specific user.
I propose to remove the limitation

@sjpadgett sjpadgett added developers This issue targets an issue that is for developers/collaborators/module writers/technical users Module Support Use if effects modules labels Jan 26, 2024
@sjpadgett sjpadgett self-assigned this Jan 26, 2024
@adunsulag adunsulag added this to the 7.0.2.1 milestone Apr 23, 2024
@adunsulag adunsulag changed the title User settings not set if not in globals. feat: User settings not set if not in globals. May 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
developers This issue targets an issue that is for developers/collaborators/module writers/technical users Module Support Use if effects modules
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants