Project

General

Profile

Actions

Bug #21429

closed

rollback button in even log can be clicked several time in a row and error are stacked in other screens

Added by François ARMAND over 1 year ago. Updated over 1 year ago.

Status:
Released
Priority:
N/A
Category:
Web - UI & UX
Target version:
Severity:
Minor - inconvenience | misleading | easy workaround
UX impact:
I dislike using that feature
User visibility:
Operational - other Techniques | Rudder settings | Plugins
Effort required:
Small
Priority:
71
Name check:
To do
Fix check:
To do
Regression:

Description

This leads to a big moment of WTH because it does not look like anything is happening, and errors are not diplayed in that screen, but as soon as you navigate to an other screen, you have tens of error "rollback can not start because one is already running".

When the rollback button is clicked, we should AT MINIMA have a notification "rollback started" and a js action that disables all rollback button of the page.
It won't prevent a milicious user to trigger several rollback by reloading the page (this is handled by #18500), but it will help a normal user to understand what is happening.


Related issues 1 (0 open1 closed)

Related to Rudder - Bug #18500: clicking twice on rollbacking event in 6.1 breaks rudderResolvedActions
Actions #1

Updated by François ARMAND over 1 year ago

  • Related to Bug #18500: clicking twice on rollbacking event in 6.1 breaks rudder added
Actions #2

Updated by Alexis Mousset over 1 year ago

  • Target version changed from 6.2.16 to 6.2.17
Actions #3

Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 6.2.17 to 997
  • Priority changed from 48 to 72
Actions #4

Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 997 to 6.2.18
Actions #5

Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 6.2.18 to 6.2.19
Actions #6

Updated by Elaad FURREEDAN over 1 year ago

  • Status changed from New to In progress
  • Assignee set to Elaad FURREEDAN
  • Priority changed from 72 to 71
Actions #7

Updated by Elaad FURREEDAN over 1 year ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Elaad FURREEDAN to Vincent MEMBRÉ
  • Pull Request set to https://github.com/Normation/rudder/pull/4463
Actions #8

Updated by Anonymous over 1 year ago

  • Status changed from Pending technical review to Pending release
Actions #9

Updated by Vincent MEMBRÉ over 1 year ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 6.2.19, 7.0.8 and 7.1.6 which were released today.

Actions

Also available in: Atom PDF