Skip to content

fix(multi-select): Forward the error prop to the MultiSelect Input #3309

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

Merged
merged 7 commits into from
May 27, 2025

Conversation

NicholasBoll
Copy link
Member

Summary

Fixes: #3296

MultiSelect.Input and MultiSelect.SearchInput now handle the error prop. Added error story and state table.

Release Category

Components


Checklist

For the Reviewer

  • PR title is short and descriptive
  • PR summary describes the change (Fixes/Resolves linked correctly)

Where Should the Reviewer Start?

Areas for Feedback? (optional)

  • Code
  • Documentation
  • Testing

@NicholasBoll NicholasBoll added the ready for review Code is ready for review label May 20, 2025
Copy link

cypress bot commented May 20, 2025

Workday/canvas-kit    Run #8862

Run Properties:  status check passed Passed #8862  •  git commit c27924c708 ℹ️: Merge c359dd968818802d8e1fedf63ac25bdfaeaaddd1 into e1242b1bb4b857e2132a89485c5d...
Project Workday/canvas-kit
Branch Review fix/multi-select-error
Run status status check passed Passed #8862
Run duration 02m 47s
Commit git commit c27924c708 ℹ️: Merge c359dd968818802d8e1fedf63ac25bdfaeaaddd1 into e1242b1bb4b857e2132a89485c5d...
Committer Nicholas Boll
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 1
Tests that did not run due to a developer annotating a test with .skip  Pending 21
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 936
View all changes introduced in this branch ↗︎
UI Coverage  21.05%
  Untested elements 1547  
  Tested elements 410  
Accessibility  99.29%
  Failed rules  6 critical   5 serious   0 moderate   2 minor
  Failed elements 98  

@mannycarrera4 mannycarrera4 added automerge and removed ready for review Code is ready for review labels May 27, 2025
@alanbsmith alanbsmith enabled auto-merge (squash) May 27, 2025 15:01
@alanbsmith alanbsmith merged commit c375b48 into Workday:support May 27, 2025
25 of 30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants