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

FEEL for Labels and Descriptions #658

Merged
merged 3 commits into from
May 26, 2023

Conversation

pinussilvestrus
Copy link
Contributor

@pinussilvestrus pinussilvestrus commented May 16, 2023

Related to #653

Adds FEEL support for label and description.

Demo: https://demo-653-labels-descriptions-o--camunda-form-playground.netlify.app/

@bpmn-io-tasks bpmn-io-tasks bot added the in progress Currently worked on label May 16, 2023
@github-actions github-actions bot temporarily deployed to demo-653-labels-descriptions-o May 16, 2023 13:46 Destroyed
@github-actions github-actions bot temporarily deployed to demo-653-labels-descriptions-o May 16, 2023 13:46 Destroyed
@pinussilvestrus pinussilvestrus changed the title Labels with expressions FEEL for Labels and Descriptions May 17, 2023
@github-actions github-actions bot temporarily deployed to demo-653-labels-descriptions-o May 17, 2023 09:24 Destroyed
@github-actions github-actions bot temporarily deployed to demo-653-labels-descriptions-o May 17, 2023 09:27 Destroyed
@github-actions github-actions bot temporarily deployed to demo-653-labels-descriptions-o May 17, 2023 09:39 Destroyed
@github-actions github-actions bot temporarily deployed to demo-653-labels-descriptions-o May 17, 2023 13:46 Destroyed
@pinussilvestrus pinussilvestrus changed the base branch from develop to 653-readonly-expressions May 17, 2023 13:55
@pinussilvestrus pinussilvestrus force-pushed the 653-readonly-expressions branch 2 times, most recently from 9a8b54d to e79c1eb Compare May 23, 2023 11:07
@github-actions github-actions bot temporarily deployed to demo-653-labels-descriptions-o May 23, 2023 11:38 Destroyed
@github-actions github-actions bot temporarily deployed to demo-653-labels-descriptions-o May 23, 2023 11:40 Destroyed
Base automatically changed from 653-readonly-expressions to develop May 25, 2023 06:14
@github-actions github-actions bot temporarily deployed to demo-653-labels-descriptions-o May 25, 2023 08:12 Destroyed
@pinussilvestrus pinussilvestrus marked this pull request as ready for review May 25, 2023 09:08
@bpmn-io-tasks bpmn-io-tasks bot added needs review Review pending and removed in progress Currently worked on labels May 25, 2023
@github-actions github-actions bot temporarily deployed to demo-653-labels-descriptions-o May 25, 2023 09:09 Destroyed
Copy link
Contributor

@christian-konrad christian-konrad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works flawlessly 👍

Copy link
Contributor

@vsgoulart vsgoulart left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't we have to update getSchemaVariables too?

@pinussilvestrus
Copy link
Contributor Author

Don't we have to update getSchemaVariables too?

We did: https://github.com/bpmn-io/form-js/pull/658/files#diff-858e82fabb9777bd462c5309760e5a499dc699b9a880787f45d6a3b6657223a9. There are also test cases at the end.

@github-actions github-actions bot temporarily deployed to demo-653-labels-descriptions-o May 26, 2023 07:29 Destroyed
@pinussilvestrus pinussilvestrus merged commit 5680cf4 into develop May 26, 2023
@pinussilvestrus pinussilvestrus deleted the 653-labels-descriptions-others branch May 26, 2023 14:47
@bpmn-io-tasks bpmn-io-tasks bot removed the needs review Review pending label May 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants