-
Notifications
You must be signed in to change notification settings - Fork 1
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
Implement deleting of users and administrative tweaks #54
base: master
Are you sure you want to change the base?
Conversation
Implement enhancements demanded by nafergo
I think we should show a dialog instead of directly deleting the user. Obviously this is an action that cannot be undone that quickly. |
Please respect the comment, then we should merge it? :) |
imho "destructive" action should have something like a "are you sure?" dialog. agree with boom1992 |
Onur, look into Bootstrap Modals, it should be relatively easy. (Just show the dialog from the html, the "Yes I'm sure" will trigger the actual function.) Nelson, nice to see you doing code reviews as well! :) we'll try to keep this going, even though at least my time is very limited these days. Did you test it yet? Found any bugs? |
Don't take my silence as ignorance. I was working on it just today but before I could test it our server went down. I am going to upload it as soon as possible. I am getting 404 at the moment :( |
No description provided.