Project

General

Profile

Actions
Copy link

Bug #22933

closed

diff on node properties in event log (at least) is broken

Added by Nicolas CHARLES 10 months ago. Updated 7 months ago.

Status:
Released
Priority:
N/A
Assignee:
Vincent MEMBRÉ
Category:
Web - UI & UX
Target version:
7.2.8
Pull Request:
https://github.com/Normation/rudder/p...
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:
123
Name check:
To do
Fix check:
Checked
Regression:
Yes

Description

  • there are object names
  • no colors red/green
  • not orered alphanumerically

Files

clipboard-202306201623-zhhia.png (207 KB) clipboard-202306201623-zhhia.png Nicolas CHARLES, 2023-06-20 16:23

Subtasks 1 (0 open — 1 closed)

Bug #22950: In rudder 7.3, we need to use npm for jsondiffpatchReleasedVincent MEMBRÉActions

Related issues 6 (0 open — 6 closed)

Related to Rudder - Bug #22946: NodeProperties value are not correctly serialized in event logReleasedNicolas CHARLESActions
Related to Rudder - User story #15148: Add search/pagination to eventlogs UIReleasedVincent MEMBRÉActions
Related to Rudder - User story #3330: Improve Diff in event logs detailsResolvedActions
Related to Rudder - Bug #21702: Event logs don't show the diff for directive updateResolvedActions
Related to Rudder - Bug #15019: Tag are not displayed with a nice diff in eventLog ReleasedVincent MEMBRÉActions
Has duplicate Rudder - Bug #20615: when change requests are enabled, there is no diff in the event details in the event logResolvedActions
Actions
Copy link
 #1

Updated by Nicolas CHARLES 10 months ago

  • Severity set to Minor - inconvenience | misleading | easy workaround
  • Priority changed from 0 to 128
Actions
Copy link
 #2

Updated by François ARMAND 10 months ago

  • Status changed from New to In progress
  • Assignee set to François ARMAND
Actions
Copy link
 #3

Updated by François ARMAND 10 months ago

  • Related to Bug #22946: NodeProperties value are not correctly serialized in event log added
Actions
Copy link
 #4

Updated by François ARMAND 10 months ago

The Quoted / SimpleConfigObject etc nonsense is due to #22946.

The missing nice diff is broken since #15148 because the returned JS is no more executed

Actions
Copy link
 #5

Updated by François ARMAND 10 months ago

Actions
Copy link
 #7

Updated by François ARMAND 10 months ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from François ARMAND to Vincent MEMBRÉ
  • Pull Request set to https://github.com/Normation/rudder/pull/4847
Actions
Copy link
 #8

Updated by Anonymous 10 months ago

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

Updated by François ARMAND 10 months ago

  • Subtask #22950 added
Actions
Copy link
 #10

Updated by Elaad FURREEDAN 10 months ago

  • Fix check changed from To do to Checked
Actions
Copy link
 #11

Updated by Vincent MEMBRÉ 10 months ago

This bug has been fixed in Rudder 7.2.8 and 7.3.3 which were released today.

Actions
Copy link
 #12

Updated by François ARMAND 8 months ago

Actions
Copy link
 #13

Updated by François ARMAND 7 months ago

  • Status changed from Pending release to Released
  • Priority changed from 128 to 123
Actions
Copy link
 #14

Updated by François ARMAND 7 months ago

  • Related to Bug #21702: Event logs don't show the diff for directive update added
Actions
Copy link
 #15

Updated by François ARMAND 7 months ago

  • Has duplicate Bug #20615: when change requests are enabled, there is no diff in the event details in the event log added
Actions
Copy link
 #16

Updated by François ARMAND 7 months ago

  • Related to Bug #15019: Tag are not displayed with a nice diff in eventLog added
Actions
Copy link

Also available in: Atom PDF