Skip to content

User injection bugfix#708

Open
theSecondBlueWizard wants to merge 3 commits intomasterfrom
user-injection-bugfix
Open

User injection bugfix#708
theSecondBlueWizard wants to merge 3 commits intomasterfrom
user-injection-bugfix

Conversation

@theSecondBlueWizard
Copy link
Contributor

Fixes some bugs so that a user is added to all intended groups, so that you don't have to delete registration requests. If a request exists, the request is accepted and a password is sent immediately. If not, the user is added and whenever they register they get their password sent to them by mail.

There is an unfortunate side-effect of a user able to make a user not being informed of this. I sent emails to everyone who I added who didn't have a user. This is inefficient, and a cleaner solution is desperately needed, though I think we can do that for the new website.

@theSecondBlueWizard
Copy link
Contributor Author

Need checkbox for is-component

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.

1 participant