Project

General

Profile

Actions
Copy link

Bug #26005

open

we can save an invalid technique (empty param name)

Bug #26005: we can save an invalid technique (empty param name)

Added by Nicolas CHARLES 12 months ago. Updated 2 days ago.

Status:
In progress
Priority:
2
Assignee:
Raphael GAUTHIER
Category:
Web - Technique editor
Target version:
8.3.7
Pull Request:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
0
Name check:
To do
Fix check:
To do
Regression:
No

Description

We can save an invalid technique, by adding a parameter, and editing the YAML so that it contain

params:
  - id: 394500e5-147f-4f91-b733-185cc3e8ec4f
    name: ''
    description: ''
    documentation: ''
    constraints:
      allow_empty: false

as a result, compilation breaks, and policy generation breaks
the save button should not be activated when we add an empty description and documentation

Updated by François ARMAND 11 months ago Actions
Copy link
 #1

  • Assignee set to Raphael GAUTHIER
  • Priority changed from To review to 2

This is already better in 8.2 with a clear error message and link to the problematic technique. Still, we need to add that check, it's an easy UX win

Updated by François ARMAND 11 months ago Actions
Copy link
 #2

  • Subject changed from we can save an invalid technique to we can save an invalid technique (empty param name)

Updated by Raphael GAUTHIER 11 months ago Actions
Copy link
 #3

  • Status changed from New to In progress

Updated by Vincent MEMBRÉ 10 months ago Actions
Copy link
 #5

  • Target version changed from 8.1.11 to 8.1.12

Updated by Alexis Mousset 9 months ago Actions
Copy link
 #6

  • Target version changed from 8.1.12 to 8.2.5

Updated by Vincent MEMBRÉ 9 months ago Actions
Copy link
 #7

  • Target version changed from 8.2.5 to 8.2.6

Updated by Vincent MEMBRÉ 6 months ago Actions
Copy link
 #8

  • Target version changed from 8.2.6 to 8.2.7

Updated by Vincent MEMBRÉ 5 months ago Actions
Copy link
 #9

  • Target version changed from 8.2.7 to 8.2.8

Updated by Félix DALLIDET 4 months ago Actions
Copy link
 #10

  • Target version changed from 8.2.8 to 8.2.9

Updated by Vincent MEMBRÉ 2 months ago Actions
Copy link
 #11

  • Target version changed from 8.2.9 to 1133

Updated by Alexis Mousset 21 days ago Actions
Copy link
 #12

  • Target version changed from 1133 to 8.3.6

Updated by Vincent MEMBRÉ 2 days ago Actions
Copy link
 #13

  • Target version changed from 8.3.6 to 8.3.7
Actions
Copy link

Also available in: PDF Atom