Project

General

Profile

Actions
Copy link

User story #3068

closed

User story #3049: When saving a Directive, we should have a warning that inform which rules are impacted by the modification

In directive, make the pop-up for impacted rules also pop for saving

Added by François ARMAND over 11 years ago. Updated about 9 years ago.

Status:
Released
Priority:
2
Assignee:
François ARMAND
Category:
Web - Config management
Target version:
2.5.0~beta1
Pull Request:
UX impact:
Suggestion strength:
User visibility:
Effort required:
Name check:
Fix check:
Regression:

Description

Change the behaviour of tha save button for directive edition to first call back the displaying of the pop-up (which seems to already be here).

The pop-up will have to be adapted to accept an other action, "save".

PR: https://github.com/Normation/rudder/pull/61

Actions
Copy link
 #1

Updated by Nicolas CHARLES over 11 years ago

  • Assignee set to Nicolas CHARLES
Actions
Copy link
 #2

Updated by Nicolas CHARLES over 11 years ago

  • Status changed from 8 to In progress

When we click on Save, a popup should appear, listing the dependant rules
If there are no dependant rules, the popup should not appear and the modification should be changed immediatly

Actions
Copy link
 #3

Updated by Nicolas CHARLES over 11 years ago

  • Status changed from In progress to 10
Actions
Copy link
 #4

Updated by Nicolas CHARLES over 11 years ago

  • Assignee changed from Nicolas CHARLES to Jonathan CLARKE

The text in the poup as been updated; and the pull request also.

Jon, can you confirm the functionnal review ?

Actions
Copy link
 #5

Updated by François ARMAND over 11 years ago

  • Status changed from 10 to Pending technical review
  • Assignee changed from Jonathan CLARKE to François ARMAND
Actions
Copy link
 #6

Updated by Jonathan CLARKE over 11 years ago

  • Target version changed from 24 to 2.5.0~beta1
Actions
Copy link
 #7

Updated by François ARMAND over 11 years ago

  • Description updated (diff)
  • Status changed from Pending technical review to 12

The RT is ok.
Updating the pull request to be able to merge back in master:
https://github.com/Normation/rudder/pull/61

Actions
Copy link
 #8

Updated by Nicolas CHARLES over 11 years ago

  • Status changed from 12 to Pending release
  • % Done changed from 0 to 100
Actions
Copy link
 #9

Updated by Anonymous over 11 years ago

Actions
Copy link
 #10

Updated by Matthieu CERDA over 11 years ago

  • Status changed from Pending release to Released
Actions
Copy link
 #11

Updated by Benoît PECCATTE about 9 years ago

  • Tracker changed from Enhancement to User story
Actions
Copy link

Also available in: Atom PDF