Skip to content
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

[Implement Tracking] Enable FEEL and Templating for more properties #587

Closed
16 of 17 tasks
Skaiir opened this issue Mar 30, 2023 · 3 comments
Closed
16 of 17 tasks

[Implement Tracking] Enable FEEL and Templating for more properties #587

Skaiir opened this issue Mar 30, 2023 · 3 comments
Assignees
Labels
enhancement New feature or request

Comments

@Skaiir
Copy link
Contributor

Skaiir commented Mar 30, 2023

This is the implementation epic for https://github.com/camunda/product-hub/issues/722

What should we do?

We have just successfully integrated the feelers syntax in the text component text property in 8.2. We want to now convert over a few properties in the application to accept this syntax or simply FEEL expressions.

This includes

This needs to be broken down properly

Why should we do it?

Change the behavior of your form reactively via simple rules, based on the context provided by process variables and other form input.

We have the technical capabilities in place, now let's enable it.

Additional context

Related to https://github.com/camunda/product-hub/issues/722
Related to this, and tightly coupled: https://github.com/camunda/product-hub/issues/918. It makes sense to start on this in parallel, to have the proper support of FEEL in readonly
Miro: https://miro.com/app/board/uXjVMMtnox0=/

@Skaiir Skaiir added enhancement New feature or request ready Ready to be worked on labels Mar 30, 2023
@Skaiir Skaiir self-assigned this Mar 30, 2023
@Skaiir Skaiir added the spring cleaning Could be cleaned up one day label Apr 17, 2023
Skaiir added a commit that referenced this issue Apr 17, 2023
Skaiir added a commit that referenced this issue Apr 18, 2023
@Skaiir Skaiir removed the spring cleaning Could be cleaned up one day label Apr 24, 2023
@pinussilvestrus pinussilvestrus changed the title Enable templating syntax across the application, where it makes sense. Enable FEEL and Templating for more properties May 2, 2023
@pinussilvestrus
Copy link
Contributor

pinussilvestrus commented May 9, 2023

A user mentioning disabled as needed for FEEL: camunda/camunda-modeler#3600 => responding that readonly even fits better 👍

@pinussilvestrus pinussilvestrus changed the title Enable FEEL and Templating for more properties [Implement Tracking] Enable FEEL and Templating for more properties May 11, 2023
@pinussilvestrus
Copy link
Contributor

pinussilvestrus commented May 15, 2023

Kickoff results

https://miro.com/app/board/uXjVMMtnox0=/

  • We consider use feelers for all feel optional string fields
  • We do not support FEEL for disabled
  • We consider a standalone docs page about templating (not only in text views)
  • @pinussilvestrus to clarify with @RomanKostka and @christian-konrad: really no tracking? => we aim to handle this via generic properties panel tracking
  • Follow up: initiate for BPMN properties (e.g. task title)
  • Discuss discouraging disabled fields (cf. Discourage the use of disabled fields #362)
  • @pinussilvestrus to clarify with @RomanKostka: how do we indicate feelers fields? => not separately, as we indicate them via the optional FEEL icon already

@pinussilvestrus
Copy link
Contributor

Implementation is done and released via [email protected], basically waiting for the next release cycle for integration into Tasklist and the Modelers.

@bpmn-io-tasks bpmn-io-tasks bot removed the ready Ready to be worked on label Jul 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants