Project

General

Profile

Actions

Bug #16234

closed

Technique import doesn't fill all info, manual fixes for known info

Added by Florian Heigl over 4 years ago. Updated almost 2 years ago.

Status:
Rejected
Priority:
N/A
Assignee:
-
Category:
Web - Technique editor
Target version:
Severity:
Minor - inconvenience | misleading | easy workaround
UX impact:
User visibility:
Getting started - demo | first install | Technique editor and level 1 Techniques
Effort required:
Priority:
42
Name check:
To do
Fix check:
To do
Regression:

Description

I tested techniques from Rudder 4 into 5 and 6.

In 5:
You get the methods imported OK, but you need to copy the method name to report component for each single step. It's a very large manual overhead for the user that could be avoided by filling it by default from the method name if it was found empty.

In 6:
You also need to fill the report component for each step, but now the method name is no longer visible. So you need to copy that from somewhere else - or prettify it, but the general problem stays that now you have to guess what you actually did in that step based on the data in the text boxes of the method. it's a bit awkward & dangerous.

The defaults for methods aren't working so i.e. for package state you need to explicitely fill "latest", "default", "default", "present"


Related issues 1 (0 open1 closed)

Related to Rudder - Bug #16316: Default values of parameters in methods cannot be used in technique editorReleasedFrançois ARMANDActions
Actions #1

Updated by Florian Heigl over 4 years ago

Just noticed that the defaults not working is a general issue in 6.x's NCF, that part is unrelated to importing.

Actions #2

Updated by Vincent MEMBRÉ over 4 years ago

  • Status changed from New to Rejected
  • Target version set to 6.0.0

Thank you a lot for reporting this issue Florian, We discussed this a little on Slack and i fixed this issue in #15509 and #16226 and now you should have less issues to import a technique into a 5.0/6.0 technique editor, at least report component should now fallback to method name if not defined or if name is empty.

I need to look at the defaults of methods though, This has not changed for a long time and should work (no value is an empty string "" and the method should treat the case of the empty string as the default value) and open an issue if you have not

I'm closing this issue (only the importing part), we will validate that this is ok before new versions are released

Actions #3

Updated by Vincent MEMBRÉ over 4 years ago

I opened https://issues.rudder.io/issues/16316 To check the behavior of default values

Actions #4

Updated by Vincent MEMBRÉ over 4 years ago

  • Related to Bug #16316: Default values of parameters in methods cannot be used in technique editor added
Actions #5

Updated by Alexis Mousset almost 2 years ago

  • Project changed from 41 to Rudder
  • Category changed from Technique editor - UI/UX to Web - Technique editor
  • Priority changed from 50 to 42
Actions

Also available in: Atom PDF