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

FIO-7514: fixed an isse where new simple conditionals do not work when condition is based on the value of resource select with object value #5429

Conversation

TanyaGashtold
Copy link
Contributor

@TanyaGashtold TanyaGashtold commented Dec 7, 2023

Link to Jira Ticket

https://formio.atlassian.net/browse/FIO-7514

Description

What changed?

When the resource select with object value (select with reference, select resource without value property or select resource with 'entire object' value property) is used in the new conditional logic, we suppose that user expects the equality of labels of the value in condition and the actual value of the select component. This PRs makes so the select component label data is saved as a value for the condition and than compared with the label data of the actual select value.

Why have you chosen this solution?

Discussed with Travis as best solution for this use case.

Dependencies

formio/formio#1664

How has this PR been tested?

Manually + added tests
All tests passed locally

Checklist:

  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (if applicable)
  • My changes generate no new warnings
  • My changes include tests that prove my fix is effective (or that my feature works as intended)
  • New and existing unit/integration tests pass locally with my changes
  • Any dependent changes have corresponding PRs that are listed above

… condition is based os the value of resource select with object value
@lane-formio
Copy link
Contributor

Can you resolve the merge conflict and I believe tests should pass now with master pulled in.

Copy link
Contributor

@brendanbond brendanbond left a comment

Choose a reason for hiding this comment

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

LGTM

@brendanbond brendanbond merged commit 4be13b8 into master Dec 18, 2023
4 checks passed
lane-formio pushed a commit that referenced this pull request Dec 22, 2023
…ls-for-select-resource-with-object-value

FIO-7514: fixed an isse where new simple conditionals do not work when condition is based on the value of resource select with object value
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants