We already have a boolean flag in Findit config to disable auto-revert, but we should also surface it to a simple UI for troopers to disable this feature in case there is fire. On that simple UI, just ask for the reason to disable this feature.
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra/+/4737ce755227e4620e198e0ca0b0922e2790390e commit 4737ce755227e4620e198e0ca0b0922e2790390e Author: Chan <chanli@chromium.org> Date: Fri Jun 23 18:36:31 2017 [Findit] Add button to disable auto revert feature Bug: 733361 Change-Id: I14fe65ef80264a27853915be39f237ddeabe8f0f Reviewed-on: https://chromium-review.googlesource.com/541039 Commit-Queue: Chan Li <chanli@chromium.org> Reviewed-by: Roberto Carrillo <robertocn@chromium.org> Reviewed-by: Jeffrey Li <lijeffrey@chromium.org> [add] https://crrev.com/4737ce755227e4620e198e0ca0b0922e2790390e/appengine/findit/templates/change_auto_revert_setting.html [add] https://crrev.com/4737ce755227e4620e198e0ca0b0922e2790390e/appengine/findit/handlers/change_auto_revert_setting.py [modify] https://crrev.com/4737ce755227e4620e198e0ca0b0922e2790390e/appengine/findit/main.py [modify] https://crrev.com/4737ce755227e4620e198e0ca0b0922e2790390e/appengine/findit/ui/elements/findit-app.html [add] https://crrev.com/4737ce755227e4620e198e0ca0b0922e2790390e/appengine/findit/handlers/test/change_auto_revert_setting_test.py
Comment 1 by chanli@chromium.org
, Jun 19 2017