User story #2493
closed
Add reasons field to Directives edit form
Added by Jean VILVER about 13 years ago.
Updated about 13 years ago.
Category:
Web - Config management
- Status changed from New to Pending technical review
- % Done changed from 0 to 100
- Target version set to 2.4.0~alpha7
- Priority changed from N/A to 1 (highest)
That seems OK, even if the commits are strange. I'm wondering why the first two are identicals...
- Status changed from Pending technical review to 10
- Category set to Web - Config management
- Status changed from 10 to Discussion
This does not seem to work as expected. I just used a freshly installed version of Rudder (so with default config parameters), edited a Directive, and saved. I got a popup saying "Your changes have been saved", but then the screen returned to the previous display and a red error message saying "The reasons must have at least 5 characters" was displayed.
This should not appear if the message is not mandatory, and should appear before the confirmation popup if it is mandatory.
That should be Ok now, this problem was fixed with #2492.
Jonathan CLARKE wrote:
This does not seem to work as expected. I just used a freshly installed version of Rudder (so with default config parameters), edited a Directive, and saved. I got a popup saying "Your changes have been saved", but then the screen returned to the previous display and a red error message saying "The reasons must have at least 5 characters" was displayed.
This should not appear if the message is not mandatory, and should appear before the confirmation popup if it is mandatory.
- Status changed from Discussion to Released
Jonathan CLARKE wrote:
This does not seem to work as expected. I just used a freshly installed version of Rudder (so with default config parameters), edited a Directive, and saved. I got a popup saying "Your changes have been saved", but then the screen returned to the previous display and a red error message saying "The reasons must have at least 5 characters" was displayed.
This should not appear if the message is not mandatory, and should appear before the confirmation popup if it is mandatory.
Jean Vilver wrote:
That should be Ok now, this problem was fixed with #2492.
This is indeed fixed, thanks Jean.
Also available in: Atom
PDF
Updated by Anonymous about 13 years ago
Updated by 
Updated by Jonathan CLARKE about 13 years ago
Updated by Jean VILVER