Project

General

Profile

Actions
Copy link

Bug #23607

closed

When a mandatory field in a technique is not defined, we can save a technique and we have a nasty error

Added by Nicolas CHARLES about 1 year ago. Updated about 1 year ago.

Status:
Released
Priority:
N/A
Assignee:
Raphael GAUTHIER
Category:
Web - Technique editor
Target version:
8.0.1
Pull Request:
https://github.com/Normation/rudder/p...
Severity:
Minor - inconvenience | misleading | easy workaround
UX impact:
I hate Rudder for that
User visibility:
Getting started - demo | first install | Technique editor and level 1 Techniques
Effort required:
Priority:
131
Name check:
To do
Fix check:
Checked
Regression:
No

Description

Files

clipboard-202310181354-k2yly.png (332 KB) clipboard-202310181354-k2yly.png Nicolas CHARLES, 2023-10-18 22:54
Actions
Copy link
 #1

Updated by François ARMAND about 1 year ago

  • Severity set to Minor - inconvenience | misleading | easy workaround
  • User visibility changed from First impressions of Rudder to Getting started - demo | first install | Technique editor and level 1 Techniques
  • Priority changed from 0 to 131

So, this is an UX bug/enhancement: we would prefer to have an the required fields known in client side so that we don't rely on backend check to get the error.

Actions
Copy link
 #2

Updated by François ARMAND about 1 year ago

Also: in error code 2, we likely want to provide the user with a better looking backend error, perhaps with the error too?

Actions
Copy link
 #3

Updated by Raphael GAUTHIER about 1 year ago

  • Status changed from New to In progress
  • Assignee set to Raphael GAUTHIER
Actions
Copy link
 #4

Updated by Raphael GAUTHIER about 1 year ago

  • Status changed from In progress to New
  • Assignee deleted (Raphael GAUTHIER)
Actions
Copy link
 #5

Updated by Raphael GAUTHIER about 1 year ago

It seems like the api does not return any constraints in the methods list

Actions
Copy link
 #6

Updated by Raphael GAUTHIER about 1 year ago

Cannot reproduced on 7.3, everything works just fine

Actions
Copy link
 #7

Updated by Alexis Mousset about 1 year ago

  • Status changed from New to In progress
  • Assignee set to Alexis Mousset
Actions
Copy link
 #8

Updated by Alexis Mousset about 1 year ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Alexis Mousset to François ARMAND
  • Pull Request set to https://github.com/Normation/rudder/pull/5118
Actions
Copy link
 #9

Updated by Alexis Mousset about 1 year ago

  • Assignee changed from François ARMAND to Raphael GAUTHIER
Actions
Copy link
 #10

Updated by Alexis Mousset about 1 year ago

  • Assignee changed from Raphael GAUTHIER to François ARMAND
Actions
Copy link
 #11

Updated by Alexis Mousset about 1 year ago

  • Assignee changed from François ARMAND to Raphael GAUTHIER
Actions
Copy link
 #12

Updated by Alexis Mousset about 1 year ago

  • Status changed from Pending technical review to Pending release
Actions
Copy link
 #13

Updated by Elaad FURREEDAN about 1 year ago

  • Fix check changed from To do to Checked
Actions
Copy link
 #14

Updated by Vincent MEMBRÉ about 1 year ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 8.0.1 which was released today.

Actions
Copy link

Also available in: Atom PDF