specification icon indicating copy to clipboard operation
specification copied to clipboard

Rename data filter properties

Open cdavernas opened this issue 1 year ago • 5 comments

What would you like to be added:

To rename data filter properties

Why is this needed:

Having properties such as stateDataFilter when defining a state, or actionDataFilter when defining an action is redudant and tedious to write. So, instead of:

...
states:
- name: MyOperationState
  type: operation
  stateDataFilter: ...
  actions:
    - name: MyAction
      actionDataFilter: ...
      ...
...

... I propose we use:

...
states:
- name: MyOperationState
  type: operation
  dataFilter: ...
  actions:
    - name: MyAction
      dataFilter: ...
      ...
...

IMHO it's less confusing, shorter and cleaner.

cdavernas avatar Sep 09 '22 13:09 cdavernas

agree but much harder to define cleanly in json schema :) looking into it to see if there is nice way

tsurdilo avatar Sep 09 '22 14:09 tsurdilo

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

github-actions[bot] avatar Oct 29 '22 00:10 github-actions[bot]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

github-actions[bot] avatar Dec 15 '22 00:12 github-actions[bot]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

github-actions[bot] avatar Jan 31 '23 00:01 github-actions[bot]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

github-actions[bot] avatar Mar 18 '23 00:03 github-actions[bot]

Closed as resolved by 1.0.0-alpha1, and therefore as part of #843

cdavernas avatar May 17 '24 08:05 cdavernas