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

Conditionally hide component #54

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

sergei-maertens
Copy link
Member

@sergei-maertens sergei-maertens commented Dec 28, 2024

  • Prototype simple case
  • Ensure that layout components (fieldset, columns) work as expected
  • Ensure that edit grids work as expected
  • Ensure that clearOnHide behaviour is respected

@sergei-maertens sergei-maertens force-pushed the feature/conditionally-hide-component branch 4 times, most recently from 8b4d71f to 8b822f9 Compare January 8, 2025 21:08
Certain behaviours are expected to match the reference implementation.

We can define stories in a way that one is renderer with our
implementation, and another is rendered with the reference Formio.js
SDK, with the versions used in the Open Forms SDK. This way, we
guarantee that our (interaction) tests are written correctly by
validating the Formio behaviour, and these same tests are then used
to validate the behaviour of our own implementation.
@sergei-maertens sergei-maertens force-pushed the feature/conditionally-hide-component branch from 8b822f9 to 55a7fbf Compare January 11, 2025 20:39
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.

1 participant