New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(variable-mapping): show target variable name #287

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
3 participants
@falko
Copy link
Member

falko commented Dec 28, 2018

Proposed Changes

  • Show target variable names instead of the index in the lists of defined variable mappings.

The index that is currently displayed is just reflecting the order in the XML and therefore meaningless for users and process engine. The target value instead is important to see which variables are set in e.g. a child or parent process. For example the current implementation would show 0 : ${true}. The changed code would display it as myVar := ${true}. The current implementation forces users to click on each entry in the list in order to see the target variable names.

feat(variable-mapping): show target variable name
Show target variable names instead of the index in the lists of defined variable mappings.

@wafflebot wafflebot bot added the needs review label Dec 28, 2018

@falko

This comment has been minimized.

Copy link
Member Author

falko commented Dec 28, 2018

@philippfromme

This comment has been minimized.

Copy link
Contributor

philippfromme commented Jan 14, 2019

Merged via 4e0ff7e.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment