Bug #21905
openHard to see why we can't save a technique in editor
Description
This is a problem, it's not easy to see what wen't wrong when we try to save a technique.
We have to define a strategy on how to provide the feedback.
Currently¶
We disable the Save button when we consider that a technique is not valid (I.E: empty technique name, empty method required value etc). We display bellow the input, an error message, but in case of a method it's not easy to find witch method is on error unless we open all method and check if there is an error message bellow the input.
And the difference between the normal and the disabled button is hardly visible, which adds to the confusion.
In addition to this, on creation, no error message appear, so it's can be hard to see if we forget to complete an input especially in methods
Error message below are trigger after a first edition of the input
I.e
What we can do¶
These are some ideas of how to improve the actual behavior, somes of this ideas are not compatible with each other, we have to decide which we want to apply, or define new ones.- add tooltips when a technique is not savable of errors messages
- add a global message in method section to indicate which methods are wrong
I.e "Method "A, B, C" are on error, please verify their input" - Never disable the save button and let error notification from API provide the error message (=> this implies that the API provide relevant error messages)
- Put error messages on creation when no inputs are filled yet
Files
Updated by Elaad FURREEDAN about 2 years ago
- Related to Bug #21903: Hard to notice if Save button is disabled in technique editor added
Updated by Vincent MEMBRÉ about 2 years ago
- Target version changed from 7.1.7 to 7.1.8
Updated by Vincent MEMBRÉ almost 2 years ago
- Target version changed from 7.1.8 to 7.1.9
Updated by Vincent MEMBRÉ almost 2 years ago
- Target version changed from 7.1.9 to 7.1.10
Updated by Vincent MEMBRÉ almost 2 years ago
- Target version changed from 7.1.10 to 1016
Updated by Benoît PECCATTE over 1 year ago
- Severity set to Minor - inconvenience | misleading | easy workaround
- User visibility set to Operational - other Techniques | Rudder settings | Plugins
- Priority changed from 0 to 60
Updated by Alexis Mousset over 1 year ago
- Target version changed from 1016 to 7.2.5
Updated by Vincent MEMBRÉ over 1 year ago
- Target version changed from 7.2.5 to 7.2.6
- Priority changed from 60 to 59
Updated by Vincent MEMBRÉ over 1 year ago
- Target version changed from 7.2.6 to 7.2.7
- Priority changed from 59 to 58
Updated by Vincent MEMBRÉ over 1 year ago
- Target version changed from 7.2.7 to 7.2.8
- Priority changed from 58 to 57
Updated by Vincent MEMBRÉ over 1 year ago
- Target version changed from 7.2.8 to 7.2.9
- Priority changed from 57 to 56
Updated by Vincent MEMBRÉ over 1 year ago
- Target version changed from 7.2.9 to 7.2.10
- Priority changed from 56 to 55
Updated by Alexis Mousset over 1 year ago
- Target version changed from 7.2.10 to 7.2.11
Updated by Vincent MEMBRÉ about 1 year ago
- Target version changed from 7.2.11 to 1046
- Priority changed from 55 to 54
Updated by Alexis Mousset about 1 year ago
- Target version changed from 1046 to 7.3.8
- Priority changed from 54 to 53
Updated by Vincent MEMBRÉ about 1 year ago
- Target version changed from 7.3.8 to 7.3.9
Updated by Vincent MEMBRÉ about 1 year ago
- Target version changed from 7.3.9 to 7.3.10
Updated by Vincent MEMBRÉ 12 months ago
- Target version changed from 7.3.10 to 7.3.11
Updated by Vincent MEMBRÉ 10 months ago
- Target version changed from 7.3.11 to 7.3.12
Updated by Vincent MEMBRÉ 9 months ago
- Target version changed from 7.3.12 to 7.3.13
Updated by Vincent MEMBRÉ 9 months ago
- Target version changed from 7.3.13 to 7.3.14
Updated by Vincent MEMBRÉ 7 months ago
- Target version changed from 7.3.14 to 7.3.15
Updated by Vincent MEMBRÉ 6 months ago
- Target version changed from 7.3.15 to 7.3.16
Updated by Vincent MEMBRÉ 5 months ago
- Target version changed from 7.3.16 to 7.3.17