-
Notifications
You must be signed in to change notification settings - Fork 1
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
BREAKING CHANGE: Remove feature class for bordered Alert #DS-1230 #1382
BREAKING CHANGE: Remove feature class for bordered Alert #DS-1230 #1382
Conversation
✅ Deploy Preview for spirit-design-system ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
Please also update the visual tests (when everything is running, start make test-e2e-update
and it should update the screenshot for alert). When you push the updated screenshot, you can add label run visual tests
to this PR to let the CI check if everything is correct.
And remove the related section from the deprecation-v2.md file in the packages/web.
ce4767f
to
1781b68
Compare
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.
Good job! :) 💪
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.
👍🏻
3b50f64
to
5318486
Compare
Here is the URL of the uploaded artifact: https://github.com/lmc-eu/spirit-design-system/actions/runs/8919815625/artifacts/1466133616 |
5318486
to
49f32c5
Compare
Description
Additional context
Issue reference