Project

General

Profile

Actions

Bug #12282

closed

In technique editor, technique with missing generic method need a warning icon

Added by François ARMAND over 6 years ago. Updated over 1 year ago.

Status:
Released
Priority:
N/A
Category:
Web - Technique editor
Target version:
Severity:
Minor - inconvenience | misleading | easy workaround
UX impact:
I dislike using that feature
User visibility:
Infrequent - complex configurations | third party integrations
Effort required:
Small
Priority:
61
Name check:
Fix check:
Checked
Regression:
No

Description

We can need to use bundle which are not in the generic method library. It is not an error to have such a bundle, but it must be differenciated visualy ("warning: user, that method does not exists in provided ncf lib, you must provide it or it will break at run time").

It could be something like that at technique level:

And an other icon should be added on generic method in technique:

Note: I'm not sure we want it to be a warning or an error.


Files

clipboard-202303021705-ugm5c.png (23.3 KB) clipboard-202303021705-ugm5c.png François ARMAND, 2023-03-02 17:05
clipboard-202303021708-reus3.png (24.5 KB) clipboard-202303021708-reus3.png François ARMAND, 2023-03-02 17:08
clipboard-202304121226-lvzx1.png (112 KB) clipboard-202304121226-lvzx1.png François ARMAND, 2023-04-12 12:26

Subtasks 1 (0 open1 closed)

Bug #22635: Remove "ncf" mentions in warning unknown methodReleasedAlexis MoussetActions
Actions #1

Updated by Vincent MEMBRÉ over 6 years ago

  • Target version changed from 4.1.11 to 4.1.12
Actions #2

Updated by Vincent MEMBRÉ over 6 years ago

  • Target version changed from 4.1.12 to 4.1.13
Actions #3

Updated by Benoît PECCATTE over 6 years ago

  • Target version changed from 4.1.13 to 411
Actions #4

Updated by Benoît PECCATTE over 6 years ago

  • Target version changed from 411 to 4.1.13
Actions #5

Updated by Vincent MEMBRÉ over 6 years ago

  • Target version changed from 4.1.13 to 4.1.14
Actions #6

Updated by Benoît PECCATTE over 6 years ago

  • Target version changed from 4.1.14 to 4.1.15
Actions #7

Updated by François ARMAND about 6 years ago

  • Subject changed from In technique editor, non existing generic method are not differenciated from the one in lib to In technique editor, technique broken (because missing generic method) are not highlighted
  • Severity set to Minor - inconvenience | misleading | easy workaround
  • User visibility set to Infrequent - complex configurations | third party integrations
  • Priority changed from 0 to 20
Actions #9

Updated by Vincent MEMBRÉ about 6 years ago

  • Target version changed from 4.1.15 to 4.1.16
Actions #10

Updated by Vincent MEMBRÉ about 6 years ago

  • Target version changed from 4.1.16 to 4.1.17
  • Priority changed from 20 to 19
Actions #11

Updated by François ARMAND about 6 years ago

  • Effort required set to Small
  • Priority changed from 19 to 34
Actions #12

Updated by Vincent MEMBRÉ almost 6 years ago

  • Target version changed from 4.1.17 to 4.1.18
  • Priority changed from 34 to 33
Actions #13

Updated by Vincent MEMBRÉ almost 6 years ago

  • Target version changed from 4.1.18 to 4.1.19
  • Priority changed from 33 to 32
Actions #14

Updated by Alexis Mousset almost 6 years ago

  • Target version changed from 4.1.19 to 4.1.20
Actions #15

Updated by François ARMAND over 5 years ago

  • Target version changed from 4.1.20 to 4.1.21
  • Priority changed from 32 to 31
Actions #16

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 4.1.21 to 4.1.22
Actions #17

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 4.1.22 to 4.1.23
Actions #18

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 4.1.23 to 4.1.24
Actions #19

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 4.1.24 to 588
Actions #20

Updated by Nicolas CHARLES over 5 years ago

  • Target version changed from 588 to 5.0.13

most likely still the case in 5.0

Actions #21

Updated by Vincent MEMBRÉ about 5 years ago

  • Target version changed from 5.0.13 to 5.0.14
Actions #22

Updated by Vincent MEMBRÉ about 5 years ago

  • Target version changed from 5.0.14 to 5.0.15
Actions #23

Updated by Vincent MEMBRÉ almost 5 years ago

  • Target version changed from 5.0.15 to 5.0.16
Actions #24

Updated by Alexis Mousset almost 5 years ago

  • Target version changed from 5.0.16 to 5.0.17
Actions #25

Updated by Vincent MEMBRÉ over 4 years ago

  • Target version changed from 5.0.17 to 5.0.18
Actions #26

Updated by Vincent MEMBRÉ over 4 years ago

  • Target version changed from 5.0.18 to 5.0.19
Actions #27

Updated by Vincent MEMBRÉ about 4 years ago

  • Target version changed from 5.0.19 to 5.0.20
Actions #28

Updated by Vincent MEMBRÉ about 4 years ago

  • Target version changed from 5.0.20 to 797
Actions #29

Updated by Benoît PECCATTE over 3 years ago

  • Target version changed from 797 to 6.1.14
  • Priority changed from 31 to 32
Actions #30

Updated by Vincent MEMBRÉ over 3 years ago

  • Target version changed from 6.1.14 to 6.1.15
Actions #31

Updated by Vincent MEMBRÉ over 3 years ago

  • Target version changed from 6.1.15 to 6.1.16
Actions #32

Updated by Vincent MEMBRÉ about 3 years ago

  • Target version changed from 6.1.16 to 6.1.17
  • Priority changed from 32 to 33
Actions #33

Updated by Vincent MEMBRÉ about 3 years ago

  • Target version changed from 6.1.17 to 6.1.18
Actions #34

Updated by Vincent MEMBRÉ almost 3 years ago

  • Target version changed from 6.1.18 to 6.1.19
Actions #35

Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 6.1.19 to 6.1.20
  • Priority changed from 33 to 34
Actions #36

Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 6.1.20 to 6.1.21
  • Priority changed from 34 to 35
Actions #37

Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 6.1.21 to old 6.1 issues to relocate

Updated by François ARMAND over 1 year ago

Actions #39

Updated by François ARMAND over 1 year ago

  • UX impact set to I dislike using that feature
  • Priority changed from 0 to 61

The case is extremelly infrequent, but encoutering it make it even more complicated to understand since there is nothing telling us that we are out of our normal way.

Actions #40

Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 7.2.5 to 7.2.6
Actions #41

Updated by Elaad FURREEDAN over 1 year ago

  • Status changed from New to In progress
  • Assignee set to Elaad FURREEDAN
Actions #42

Updated by Elaad FURREEDAN over 1 year ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Elaad FURREEDAN to François ARMAND
  • Pull Request set to https://github.com/Normation/rudder/pull/4737
Actions #44

Updated by Anonymous over 1 year ago

  • Status changed from Pending technical review to Pending release
Actions #45

Updated by Vincent MEMBRÉ over 1 year ago

  • Subtask #22635 added
Actions #46

Updated by Vincent MEMBRÉ over 1 year ago

  • Fix check set to Checked
Actions #47

Updated by Vincent MEMBRÉ over 1 year ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 7.2.6 and 7.3.1 which were released today.

Actions

Also available in: Atom PDF