flipper icon indicating copy to clipboard operation
flipper copied to clipboard

Confirmation for disabling feature doesn't always trigger

Open rstawarz opened this issue 1 month ago • 1 comments

We had an issue in production where someone accidentally clicked the disable button when they shouldn't have. They reported there was no confirmation shown to warn them. I was able to recreate the issue in various scenarios. I think there's an issue with how the javascript enables the confirmation. I'm guessing there's an issue with the JS being applied the button. A couple suggestions:

  • The button should be disabled until the confirmation click handler is enabled.
  • the confirmation_text could be part of the submission to deactivating the flag to ensure this scenario can't occur.

https://github.com/user-attachments/assets/211422dd-c2db-4613-8e56-2f8751bae5bf

rstawarz avatar Nov 20 '25 15:11 rstawarz

I'll look into this. Thanks for the detailed report!

jnunemaker avatar Nov 20 '25 16:11 jnunemaker

@rstawarz should have this fix in next release this week.

jnunemaker avatar Dec 08 '25 15:12 jnunemaker