Project

General

Profile

Actions

Bug #22895

closed

When adding a directive to a rule we cannot add event log anymore

Added by Benoît PECCATTE over 1 year ago. Updated about 1 year ago.

Status:
Resolved
Priority:
N/A
Category:
Web - UI & UX
Target version:
Severity:
Critical - prevents main use of Rudder | no workaround | data loss | security
UX impact:
I hate Rudder for that
User visibility:
Operational - other Techniques | Rudder settings | Plugins
Effort required:
Priority:
141
Name check:
To do
Fix check:
To do
Regression:
Yes

Description

There was a log popup before, this is not the case anymore, it is not possible to provide a specific log message
And in change validation mode there is no message telling a change request has been created

EDIT: techniques in editor need the same reason pop-up on validation (worked-around in #23000 but it's not a correction)


Files

clipboard-202307240901-ep2un.png (27.9 KB) clipboard-202307240901-ep2un.png François ARMAND, 2023-07-24 09:01

Related issues 3 (0 open3 closed)

Related to Rudder - Bug #23000: When changes messages are mandatory and saving a technique we have an error message about missing info but technique is savedReleasedVincent MEMBRÉActions
Related to Rudder - Bug #23372: Add a way to set a message if "change audit logs" setting is enabledReleasedRaphael GAUTHIERActions
Related to Rudder - Enhancement #21931: Display the list of pending change requests related to the selected ruleReleasedVincent MEMBRÉActions
Actions #1

Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 7.3.3 to 7.3.4
Actions #2

Updated by François ARMAND over 1 year ago

  • Severity set to Critical - prevents main use of Rudder | no workaround | data loss | security
  • UX impact set to I hate Rudder for that
  • User visibility set to Operational - other Techniques | Rudder settings | Plugins
  • Priority changed from 0 to 148
  • Regression changed from No to Yes
Actions #3

Updated by François ARMAND over 1 year ago

  • Description updated (diff)
Actions #4

Updated by François ARMAND over 1 year ago

  • Related to Bug #23000: When changes messages are mandatory and saving a technique we have an error message about missing info but technique is saved added
Actions #5

Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 7.3.4 to 7.3.5
  • Priority changed from 148 to 147
Actions #6

Updated by François ARMAND over 1 year ago

This is a missing feature (a regression) from when we ported the rule dashboard to elm. It needs to be developped:

The strict minimum and most important:
- a pop-up with a message (optionnal or required based on the parameter value in rudder settings)

That exact same feature need to be done in the technique editor to correct #23000 (where only a workaround was implemented).

Pop-up looks like that in directives:

It can be something else than a pop-up if UX is better, but the primary goal is to bring back the feature.

Should also be reimplemented for rule:
- if the change validation plugin is enabled, the list of active change requests for the currently viewed rule

Actions #7

Updated by Alexis Mousset over 1 year ago

  • Target version changed from 7.3.5 to 7.3.6
  • Priority changed from 147 to 146
Actions #8

Updated by Raphael GAUTHIER over 1 year ago

  • Related to Bug #23372: Add a way to set a message if "change audit logs" setting is enabled added
Actions #9

Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 7.3.6 to 7.3.7
  • Priority changed from 146 to 143
Actions #10

Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 7.3.7 to 7.3.8
Actions #11

Updated by Raphael GAUTHIER about 1 year ago

  • Status changed from New to Resolved
  • Priority changed from 143 to 141

I'm closing this ticket since it has been fixed in #23372

Actions #12

Updated by François ARMAND about 1 year ago

  • Related to Enhancement #21931: Display the list of pending change requests related to the selected rule added
Actions

Also available in: Atom PDF