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.

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

Make reason mandatory to reject a new user? #1441

Closed
yarikoptic opened this issue Jan 20, 2023 · 2 comments · Fixed by #1443
Closed

Make reason mandatory to reject a new user? #1441

yarikoptic opened this issue Jan 20, 2023 · 2 comments · Fixed by #1443
Assignees
Labels
released This issue/pull request has been released. UX Affects usability of the system

Comments

@yarikoptic
Copy link
Member

#1301 added rejection reason. I often click on Reject for some reason assuming it would somehow give me a dialog to provide a reason... sure thing it doesn't and thus I reject without stating a reason. I think it is reasonable to expect to provide reason for every rejection, thus should we make Reject disabled unless reason is specified or add some check to not actually do the action and give error message if no reason is given?

@yarikoptic yarikoptic added the UX Affects usability of the system label Jan 20, 2023
@waxlamp
Copy link
Member

waxlamp commented Jan 23, 2023

@mvandenburgh can you work on this? I think it should be pretty simple--if it's not, please report back here.

@dandibot
Copy link
Member

🚀 Issue was released in v0.3.12 🚀

@dandibot dandibot added the released This issue/pull request has been released. label Jan 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released This issue/pull request has been released. UX Affects usability of the system
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants