-
Notifications
You must be signed in to change notification settings - Fork 3
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
Disallow users from making requests until first/last name are filled in #605
Comments
@helbashandy I think this would make a simple first issue to let you see/touch much of the codebase. Most views in It's probably worth considering placing checks like these (agreement being signed, names being filled, etc.) into a new mixin to avoid duplicate code. I also want to document which views require what. Whether that belongs only in the code or has a place in our separate docs repo is a conversation for us to have. |
* Adds a permissions.py file as an initial step to centeralize permissions across the app. * Adds first and last name check to the test_func on ProjectRequestView. * Creates a generic wrapper decorator to be used around test_func to allow gradual progressive refactor of test_func towards a more centeralized and modular permissions management solution. closes #605
@helbashandy Some additional relevant views:
|
* Adds a permissions.py file as an initial step to centeralize permissions across the app. * Adds first and last name check to the test_func on requests to create or join a project. * Creates a generic wrapper decorator to be used around test_func to allow gradual progressive refactor of test_func towards a more centralized and modular permissions management solution. * Adds test for request creation Fixes #605
CILogon is given first/last name information by the identity provider chosen by the user. In rare cases, the provider doesn't give one or the other of the names, so we have incomplete information.
In cases like these, users should be prevented from making most (all?) requests (especially requests to join/be added to a project and, by extension, gain access to the cluster) until they fill in both first/last names from the user profile page.
We already have checks for whether the user agreement is signed. Consider including checks like these in some kind of utility function/mixin to avoid code duplication.
It might also be useful to include a banner upon initial authentication alerting the user to do this.
The text was updated successfully, but these errors were encountered: