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

Feature request: skip disabled users #56

Open
stdoma opened this issue Apr 12, 2023 · 3 comments
Open

Feature request: skip disabled users #56

stdoma opened this issue Apr 12, 2023 · 3 comments

Comments

@stdoma
Copy link

stdoma commented Apr 12, 2023

Disabled users are also sorted into global groups if the rules match.
It would be nice if you could determine in the plugin or in the individual rules that deactivated users are not assigned.

I implemented a general restriction in the plugin myself. - Thanks for the good setup.
This may also be relevant for other people.

@abias
Copy link
Member

abias commented Apr 30, 2023

Hi @stdoma ,

in #27 / #21, a patch was just integrated which stops the plugin from adding deleted users to cohorts. This is not the same as disabled (= suspended) users, but it's definitely something which has been missing up to now.

Having said that, I would disagree that this plugin should not add suspended users to cohorts. Moodle treats suspended users like active users in several places, especially in participants lists within courses.

Against this background, I would vote against this feature request, but I will leave it open for some time for others to comment.

Cheers,
Alex

@ChrisCYC
Copy link

ChrisCYC commented Jul 18, 2023

I'd second to be able to specify whether the plugin includes or excludes disabled users. This could just be via a toggle in the settings. I have an issue at the moment that when users leave the orgnisation, I need to manually enter their record and remove a field so they no longer pull into the cohort. To automate cohort removal based upon whether a Moodle user is 'active' would be great!

@jBurson-13
Copy link

This is an awesome plugin. I think it would be great to have the ability to exclude/remove users from the cohort based on the users active/suspended status.

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

No branches or pull requests

4 participants