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.

Sign up for GitHub

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

Jump to bottom

Block password reset for Globus confined users #1393

Open
wants to merge 9 commits into
base: dev
Choose a base branch
from
Open

Block password reset for Globus confined users #1393

wants to merge 9 commits into from

Conversation

MontrealSergiy
Copy link
Contributor

@MontrealSergiy MontrealSergiy self-assigned this May 14, 2024
@MontrealSergiy MontrealSergiy marked this pull request as draft May 14, 2024 13:21
@MontrealSergiy MontrealSergiy changed the base branch from master to dev May 14, 2024 13:21
@MontrealSergiy MontrealSergiy requested a review from prioux May 14, 2024 17:00
@MontrealSergiy MontrealSergiy marked this pull request as ready for review May 14, 2024 17:01
@MontrealSergiy MontrealSergiy linked an issue May 14, 2024 that may be closed by this pull request
A user with a mandatory globus login can ask for a password reset (and they shouldn't). #1391
Open
@MontrealSergiy
Copy link
Contributor Author

accepted, note a few minor changes to NeuroHub users controllers added

@natacha-beck
Copy link
Contributor

Re-tested, works fine.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@natacha-beck natacha-beck natacha-beck approved these changes

@prioux prioux Awaiting requested review from prioux

Assignees

@MontrealSergiy MontrealSergiy

Labels
Bug Priority: Normal User Interface
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

A user with a mandatory globus login can ask for a password reset (and they shouldn't).
2 participants
@MontrealSergiy @natacha-beck