Project

General

Profile

Actions
Copy link

Enhancement #21931

closed

Display the list of pending change requests related to the selected rule

Added by Elaad FURREEDAN almost 3 years ago. Updated over 1 year ago.

Status:
Released
Priority:
N/A
Assignee:
Vincent MEMBRÉ
Category:
Web - UI & UX
Target version:
7.3.11
Pull Request:
https://github.com/Normation/rudder/p...
UX impact:
I dislike using that feature
Suggestion strength:
User visibility:
Operational - other Techniques | Rudder settings | Plugins
Effort required:
Large
Name check:
To do
Fix check:
Checked
Regression:
Yes

Description

Change request in rule's page are managed very badly. It's hard to know what happen when we try to edit a rule with the plugin change request enable.
This is a regression since we port the page in Elm in 7.0

Currently

When change request plugin is enabled, if we edit a rule, a change request is generated, but we don't have any information on the rule that there is a pending CR.
We have a success notification, we can see for example that the title of the rule change, but not in the tree. In fact nothing has changed, but the notification misleads us.

What we can do

We can take the operation in Group's page:
  • Display a message on top to see all CR pending to the rule
  • Do not update any information on save
  • No notification on save

Files

Download all files
rule_no_CR_popup.gif (267 KB) rule_no_CR_popup.gif Elaad FURREEDAN, 2022-10-07 16:20
clipboard-202210071624-z8tqz.png (357 KB) clipboard-202210071624-z8tqz.png Elaad FURREEDAN, 2022-10-07 16:25

Subtasks 2 (0 open — 2 closed)

Bug #24082: Missing title for pending change request in rule pageReleasedElaad FURREEDANActions
Enhancement #24088: Don't do REST call if change request plugin is not enabledReleasedVincent MEMBRÉActions

Related issues 5 (0 open — 5 closed)

Related to Rudder - Bug #22895: When adding a directive to a rule we cannot add event log anymoreResolvedRaphael GAUTHIERActions
Related to Rudder - Bug #23372: Add a way to set a message if "change audit logs" setting is enabledReleasedRaphael GAUTHIERActions
Related to Rudder plugins - Enhancement #23907: List change requests with additional filtersReleasedFrançois ARMANDActions
Related to Rudder - Bug #24079: Fix small ui error in the rules information tabReleasedVincent MEMBRÉActions
Has duplicate Rudder - Enhancement #23676: Display the list of pending change requests related to the selected ruleRejectedRaphael GAUTHIERActions
Actions
Copy link
 #1

Updated by Vincent MEMBRÉ almost 3 years ago

  • Target version changed from 7.1.7 to 7.1.8
Actions
Copy link
 #2

Updated by François ARMAND over 2 years ago

  • Severity set to Minor - inconvenience | misleading | easy workaround
  • User visibility set to Operational - other Techniques | Rudder settings | Plugins
  • Priority changed from 0 to 64
Actions
Copy link
 #3

Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 7.1.8 to 7.1.9
  • Priority changed from 64 to 63
Actions
Copy link
 #4

Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 7.1.9 to 7.1.10
  • Priority changed from 63 to 62
Actions
Copy link
 #5

Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 7.1.10 to 1016
  • Priority changed from 62 to 61
Actions
Copy link
 #6

Updated by François ARMAND over 2 years ago

  • Target version changed from 1016 to 7.2.5
Actions
Copy link
 #7

Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 7.2.5 to 7.2.6
  • Priority changed from 61 to 59
Actions
Copy link
 #8

Updated by Elaad FURREEDAN over 2 years ago

  • Effort required changed from Medium to Large
  • Priority changed from 59 to 47
Actions
Copy link
 #9

Updated by Vincent MEMBRÉ about 2 years ago

  • Target version changed from 7.2.6 to 7.2.7
  • Priority changed from 47 to 46
Actions
Copy link
 #10

Updated by Vincent MEMBRÉ about 2 years ago

  • Target version changed from 7.2.7 to 7.2.8
Actions
Copy link
 #11

Updated by Vincent MEMBRÉ about 2 years ago

  • Target version changed from 7.2.8 to 7.2.9
  • Priority changed from 46 to 44
Actions
Copy link
 #12

Updated by Vincent MEMBRÉ almost 2 years ago

  • Target version changed from 7.2.9 to 7.2.10
Actions
Copy link
 #13

Updated by Alexis Mousset almost 2 years ago

  • Target version changed from 7.2.10 to 7.2.11
  • Priority changed from 44 to 43
Actions
Copy link
 #14

Updated by Vincent MEMBRÉ almost 2 years ago

  • Target version changed from 7.2.11 to 1046
  • Priority changed from 43 to 42
Actions
Copy link
 #15

Updated by Raphael GAUTHIER almost 2 years ago

  • Subject changed from Missing integration of change request in rule page to Display the list of pending change requests related to the selected rule
Actions
Copy link
 #16

Updated by Raphael GAUTHIER almost 2 years ago

  • Target version changed from 1046 to 7.3.8
Actions
Copy link
 #17

Updated by François ARMAND almost 2 years ago

  • Related to Bug #22895: When adding a directive to a rule we cannot add event log anymore added
  • Related to Bug #23372: Add a way to set a message if "change audit logs" setting is enabled added
Actions
Copy link
 #18

Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 7.3.8 to 7.3.9
Actions
Copy link
 #19

Updated by Raphael GAUTHIER over 1 year ago

  • Has duplicate Enhancement #23676: Display the list of pending change requests related to the selected rule added
Actions
Copy link
 #20

Updated by Raphael GAUTHIER over 1 year ago

  • Tracker changed from Bug to Enhancement
  • Severity deleted (Minor - inconvenience | misleading | easy workaround)
  • Priority deleted (42)
Actions
Copy link
 #21

Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 7.3.9 to 7.3.10
Actions
Copy link
 #22

Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 7.3.10 to 7.3.11
Actions
Copy link
 #23

Updated by Raphael GAUTHIER over 1 year ago

  • Target version changed from 7.3.11 to 8.0.5
Actions
Copy link
 #24

Updated by Raphael GAUTHIER over 1 year ago

  • Status changed from New to In progress
  • Assignee set to Raphael GAUTHIER
Actions
Copy link
 #25

Updated by Clark ANDRIANASOLO over 1 year ago

Actions
Copy link
 #28

Updated by Raphael GAUTHIER over 1 year ago

  • Target version changed from 8.0.5 to 7.3.11
Actions
Copy link
 #30

Updated by Raphael GAUTHIER over 1 year ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Raphael GAUTHIER to Vincent MEMBRÉ
  • Pull Request set to https://github.com/Normation/rudder/pull/5330
Actions
Copy link
 #31

Updated by Raphael GAUTHIER over 1 year ago

  • Status changed from Pending technical review to Pending release
Actions
Copy link
 #32

Updated by Raphael GAUTHIER over 1 year ago

  • Related to Bug #24079: Fix small ui error in the rules information tab added
Actions
Copy link
 #33

Updated by Elaad FURREEDAN over 1 year ago

  • Related to Bug #24082: Missing title for pending change request in rule page added
Actions
Copy link
 #34

Updated by Elaad FURREEDAN over 1 year ago

  • Fix check changed from To do to Error - Blocking
Actions
Copy link
 #35

Updated by François ARMAND over 1 year ago

  • Subtask #24088 added
Actions
Copy link
 #36

Updated by Vincent MEMBRÉ over 1 year ago

  • Related to deleted (Bug #24082: Missing title for pending change request in rule page)
Actions
Copy link
 #37

Updated by Vincent MEMBRÉ over 1 year ago

  • Subtask #24082 added
Actions
Copy link
 #38

Updated by Clark ANDRIANASOLO over 1 year ago

  • Fix check changed from Error - Blocking to Checked
Actions
Copy link
 #39

Updated by Vincent MEMBRÉ over 1 year ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 7.3.11 which was released today.

Actions
Copy link

Also available in: Atom PDF