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

Feature request: CRUD confirmation #59

Closed
Pukkahq opened this issue Oct 18, 2021 · 4 comments
Closed

Feature request: CRUD confirmation #59

Pukkahq opened this issue Oct 18, 2021 · 4 comments
Labels
enhancement New feature or request Planned

Comments

@Pukkahq
Copy link

Pukkahq commented Oct 18, 2021

Is your feature request related to a problem? Please describe.
User List butons

Describe the solution you'd like
Make spacing on buttons wider
Add confimation before running command

@Pukkahq Pukkahq changed the title Feature request: Feature request: User list Oct 18, 2021
@dbeta
Copy link
Contributor

dbeta commented Oct 18, 2021

I second the confirmation. It may seem annoying to over confirm, but those buttons are primed for misclicks, and the are disruptive actions. Resetting user passwords, setting disabling user accounts, and converting to a shared mailbox are quite disruptive. Last thing I would want is a misclick by a technician interrupting a client's day.

@KelvinTegelaar
Copy link
Owner

I am notoriously anti-confirmation. I've always been a non-conformist now that I think of it.

We'll actually leave it to the community: If you want ugly and mean confirmation pop-ups that delay your work and tell you you smell of potatoes add a smile to @dbeta's comment.

If you don't want confirmation and love puppies add one to mine.

@KelvinTegelaar KelvinTegelaar added the enhancement New feature or request label Oct 18, 2021
@dbeta
Copy link
Contributor

dbeta commented Oct 18, 2021

Perhaps a compromise. If a user or system wide option to toggle "basic confirmations". It's probably more work than it sounds like, but I think it would satisfy everybody, especially as long as we could set it as default for all users.

@KelvinTegelaar KelvinTegelaar changed the title Feature request: User list Feature request: CRUD confirmation Oct 20, 2021
@KelvinTegelaar
Copy link
Owner

Added confirmations to the API Links that missed it, that does not mean everywhere in the app, but the shortlinks for sure. :)

KelvinTegelaar pushed a commit that referenced this issue Jul 18, 2024
[pull] dev from KelvinTegelaar:dev
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Planned
Projects
None yet
Development

No branches or pull requests

3 participants