Project

General

Profile

Actions

Bug #26206

closed

Directive will not be displayed when clicking on a technique then on directive in tree

Added by Clark ANDRIANASOLO 2 months ago. Updated about 1 month ago.

Status:
Released
Priority:
1 (highest)
Category:
Web - UI & UX
Target version:
Severity:
Trivial - no functional impact | cosmetic
UX impact:
It bothers me each time
User visibility:
First impressions of Rudder
Effort required:
Very Small
Priority:
117
Name check:
To do
Fix check:
Checked
Regression:
No

Description

In the directives tree, when clicking on a directive, the URL changes to one with a hash : /secure/configurationManager/directiveManagement#{"directiveId":"b6362db9-4cb0-415a-87aa-574ccf911725"}.
Then when clicking on a technique (e.g. the technique of the directive or any other one), the technique is displayed, but the URL does not change.
So, when clicking again on the same directive, the directive will not be displayed back, the technique remains displayed.
However clicking on the directive attempts to set the URL to the same as the current one, this may be helpful for solving the issue :

this.window.location.hash = "#" + JSON.stringify({"directiveId":"c65e1edc-2b23-4215-a719-c36cfd919252"});

Workaround is to refresh the page, or open other directives to change the URL, then open the current one.

Actions #1

Updated by François ARMAND 2 months ago

  • Assignee set to Clark ANDRIANASOLO
  • Priority changed from To review to 1 (highest)

Putting it to P1 because perhaps it's a quick win: thanks for the qualification!
All we have to do to correct it is to change the URL when clicking on a technique (even if we don't use the hash code to load the technique related UI).
So just "cleaning" the window location should be enough.

Can you take a look? If the proposed correction does not work, no need to spend more time on it at P1: we will change the prio to P2.

Actions #2

Updated by Clark ANDRIANASOLO 2 months ago

  • Status changed from New to In progress
Actions #3

Updated by Clark ANDRIANASOLO 2 months ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Clark ANDRIANASOLO to François ARMAND
  • Pull Request set to https://github.com/Normation/rudder/pull/6140
Actions #4

Updated by Clark ANDRIANASOLO 2 months ago

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

Updated by Elaad FURREEDAN about 1 month ago

  • Priority changed from 118 to 117
  • Fix check changed from To do to Checked
Actions #6

Updated by Vincent MEMBRÉ about 1 month ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 8.1.12 and 8.2.5 which were released today.

Actions

Also available in: Atom PDF