-
-
/trunk/reviewboard/admin/forms.py (Diff revision 2) If this only affects builtin auth, shouldn't this be part of an auth_builtin fieldset? That way it would appear/disappear based on what authentication backend you select.
-
/trunk/reviewboard/templates/admin/general_settings.html (Diff revision 2) And you could get rid of this special case.
Add support for disabling user registration.
Review Request #760 — Created March 13, 2009 and submitted
Information | |
---|---|
chipx86 | |
Review Board SVN (deprecated) | |
trunk | |
Reviewers | |
reviewboard | |
Add support for disabling user registration. This provides a checkbox in the General Settings page for disabling user registration. This only takes effect when using standard authentication, and will hide the "Register" link and redirect requests made to /account/register/ to the /account/login/ page instead.
Tested the /account/register/ path and the existence of the link with the following combinations: * Registration enabled and standard auth * Registration disabled and standard auth * Registration enabled and LDAP * Registration disabled and LDAP All cases work. Note that LDAP is a good enough test for non-standard auth types, since the only auth-specific checks are against "builtin".
Change Summary:
Move "Enable registration" into "Basic Authentication Settings"
Diff: |
Revision 3 (+24 -6) |
---|