-
Notifications
You must be signed in to change notification settings - Fork 291
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
Have an option to clear out teams and users when resetting game #526
Comments
Option added |
Legend! Thank you - much needed! |
Admin accounts will remain. Do you see any error in the logs? |
Pushed a fix I hope - I had tried to persist the admin's session since I clear the memcached after the reset to kick off all users. However, I guess that caused an issue. So the admin will have to log back in after the reset. |
Yup that fixed it! Nice one. Just one observation though... as this is a one way action with no undo button, I would perhaps suggest we have a confirmation box come up - Are you sure you want to RESET / Delete Game? and have the user enter a physical word in. As I said, there is no undo button :-). If not a physical word, perhaps just a yes no button? :-) |
I tend to find over-proofing annoying, so I just added a confirmation yes no. If you go through the process to click the reset page, click the reset button, then click confirm - that's on you. lol |
Ha brilliant! Thank you. Tend to agree, but we also have ‘dumb’ users too – “problem between keyboard and chair” type people
From: ElJeffe ***@***.***>
Date: Wednesday, 26 October 2022 at 13:53
To: moloch--/RootTheBox ***@***.***>
Cc: PJ Norris ***@***.***>, Author ***@***.***>
Subject: [EXT.MSG] Re: [moloch--/RootTheBox] Have an option to clear out teams and users when resetting game (Issue #526)
I tend to find over-proofing annoying, so I just added a confirmation yes no. If you go through the process to click the reset page, click the reset button, then click confirm - that's on you. lol
—
Reply to this email directly, view it on GitHub<https://github.com/moloch--/RootTheBox/issues/526#issuecomment-1291989721>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AZNZST5NLK3MMGNRQWKHFKLWFESWJANCNFSM6AAAAAARLSCN2E>.
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
Is is possible to have a checkbox on this screen:
that will delete all player names and team names? Please?
The text was updated successfully, but these errors were encountered: