-
Notifications
You must be signed in to change notification settings - Fork 142
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
chore(FilterPanel): deprecate filter panel and its sub components #6431
chore(FilterPanel): deprecate filter panel and its sub components #6431
Conversation
✅ Deploy Preview for carbon-for-ibm-products ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for ibm-products-web-components ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #6431 +/- ##
==========================================
+ Coverage 79.61% 79.62% +0.01%
==========================================
Files 394 394
Lines 12871 12878 +7
Branches 4268 4268
==========================================
+ Hits 10247 10254 +7
Misses 2624 2624
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good, just one minor thing!
packages/ibm-products/src/components/FilterPanel/FilterPanel.mdx
Outdated
Show resolved
Hide resolved
these are the initial snapshots, so 168 visual changes is expected, and must be approved in percy dashboard, until we merge any pr with approved visual changes, they keep appearing. |
…ucts into deplicateFilterPanel
5fb804a
Closes #6320
What did you change?
Annotated deprecation notice in Filter panel and its sub stories
How did you test and verify your work?
storybook
yarn test