-
Notifications
You must be signed in to change notification settings - Fork 10
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
Present dialog if attempting to remove self from dandiset #1125
Conversation
de33c69
to
eefe94f
Compare
I think the intent isn't to completely remove the ability for one to remove themselves as an owner, but that there should be some sort of confirmation prompt warning them before they go through with it. |
This. I thought the title of #754 made this clear but perhaps not--if we need more discussion let's have it on that issue. |
eefe94f
to
988a9b2
Compare
d64685b
to
c1c7015
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, just some suggested style changes to keep the UI consistent with the existing DLP styling
Co-authored-by: Mike VanDenburgh <37340715+mvandenburgh@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
🚀 PR was released in |
Closes #754