Project

General

Profile

Actions
Copy link

Bug #26005

open

we can save an invalid technique (empty param name)

Added by Nicolas CHARLES about 1 month ago. Updated 22 days ago.

Status:
In progress
Priority:
2
Assignee:
Raphael GAUTHIER
Category:
Web - Technique editor
Target version:
8.1.11
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

Actions
Copy link
 #1

Updated by François ARMAND about 1 month ago

  • 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

Actions
Copy link
 #2

Updated by François ARMAND about 1 month ago

  • Subject changed from we can save an invalid technique to we can save an invalid technique (empty param name)
Actions
Copy link
 #3

Updated by Raphael GAUTHIER 26 days ago

  • Status changed from New to In progress
Actions
Copy link

Also available in: Atom PDF