search

ubccr / mokey

FreeIPA self-service account management portal
BSD 3-Clause "New" or "Revised" License
194 stars 46 forks source link

Implement admin verification requirement #130

Closed cmd-ntrf closed 1 year ago

cmd-ntrf commented 1 year ago

Fix #121.

Marking this as draft as I haven't the chance to try it. The only thing I know for now is that it compiles.

The name of the parameter accounts.require_admin_verify is also up for discussion.

Contrary to previous version of Mokey, the email verification and admin verification are no longer mutually exclusive. The user can still verify its email, but if accounts.require_admin_verify is true, the account will not be enabled after the verification.

aebruno commented 1 year ago

The name of the parameter accounts.require_admin_verify is also up for discussion.

This works for me. I like it and think it makes sense to have it under accounts.

Contrary to previous version of Mokey, the email verification and admin verification are no longer mutually exclusive. The user can still verify its email, but if accounts.require_admin_verify is true, the account will not be enabled after the verification.

Sounds good :+1:

cmd-ntrf commented 1 year ago

Ready for review.