Skip to content

FIO-10415: Update select's choices classes to be compatible with choices 11#264

Merged
lane-formio merged 1 commit intomasterfrom
fio-10415
Aug 25, 2025
Merged

FIO-10415: Update select's choices classes to be compatible with choices 11#264
lane-formio merged 1 commit intomasterfrom
fio-10415

Conversation

@blakekrammes
Copy link
Contributor

@blakekrammes blakekrammes commented Aug 22, 2025

Link to Jira Ticket

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

Description

What changed?

Set choices.js class names as an array instead of a concatenated string.

Why have you chosen this solution?

Clean and simple.

Breaking Changes / Backwards Compatibility

This change will break @formio/js < 5.2.1, which is when choices.js 11 was introduced. I added a peer dependency version for @formio/js to indicate compatibility.

Dependencies

@formio/js >= 5.2.1

How has this PR been tested?

With my two hands.

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

@blakekrammes blakekrammes marked this pull request as draft August 22, 2025 18:17
@blakekrammes blakekrammes force-pushed the fio-10415 branch 2 times, most recently from 7471b79 to 41e30d0 Compare August 22, 2025 18:50
@blakekrammes blakekrammes marked this pull request as ready for review August 22, 2025 18:50
@blakekrammes blakekrammes force-pushed the fio-10415 branch 4 times, most recently from a00530a to 77aa456 Compare August 22, 2025 19:33
steps:
- name: Download build artifact
uses: actions/download-artifact@v3
uses: actions/download-artifact@v4
Copy link
Contributor Author

@blakekrammes blakekrammes Aug 22, 2025

Choose a reason for hiding this comment

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

v3 was deprecated and the publish job wouldn't run

@lane-formio lane-formio merged commit 082fad9 into master Aug 25, 2025
3 of 4 checks passed
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.

2 participants