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

#137 Plugin setting session_roles in plugin namespace not being used. #138

Merged
merged 1 commit into from
Feb 15, 2024

Conversation

joshwillcock
Copy link
Contributor

@joshwillcock joshwillcock commented Feb 15, 2024

In issue #116 the plugin settings were moved from the global namespace to the plugin namespace.

When checking for available settings roles it is checking the old $CFG location meaning any saved results are not loaded.

Simple/quick fix attached.

Thanks.

Josh

@danmarsden danmarsden merged commit 1278894 into catalyst:MOODLE_400_STABLE Feb 15, 2024
7 checks passed
@danmarsden
Copy link
Member

Thanks @joshwillcock !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants