solidus
solidus copied to clipboard
[Admin][UI] Allow select to include blank option
[!NOTE] Split from #6228
Summary
Updates UI select component to allow including blank option. Drops support for passing options as html safe string, forcing choices to be an Array, and updates few places where options were still passed as strings.
Checklist
Check out our PR guidelines for more details.
The following are mandatory for all PRs:
- [x] I agree that my PR will be published under the same license as Solidus.
- [x] I have written a thorough PR description.
- [x] I have kept my commits small and atomic.
- [x] I have used clear, explanatory commit messages.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 88.95%. Comparing base (
ffa13fc) to head (9e32dad).
Additional details and impacted files
@@ Coverage Diff @@
## main #6279 +/- ##
==========================================
- Coverage 88.95% 88.95% -0.01%
==========================================
Files 860 860
Lines 18423 18419 -4
==========================================
- Hits 16388 16384 -4
Misses 2035 2035
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
:rocket: New features to boost your workflow:
- :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
- :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.