Skip to content
New issue

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

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

Already on GitHub? # to your account

FIO-8690: Use @formio/core conditionals process for conditionals logic #5890

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

mikekotikov
Copy link
Contributor

@mikekotikov mikekotikov commented Oct 31, 2024

Link to Jira Ticket

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

Description

What changed?

Adopted the use of formio/core conditionals process to calculate conditions

Why have you chosen this solution?

Use this section to justify your choices

Breaking Changes / Backwards Compatibility

The tests do pass, but there's a chance some not covered scenarios may fail

Dependencies

formio/core

How has this PR been tested?

Use this section to describe how you tested your changes; if you haven't included automated tests, justify your reasoning

Checklist:

  • I have completed the above PR template
  • 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

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant