Skip to content
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

Merged

Conversation

curdaj
Copy link
Contributor

@curdaj curdaj commented Apr 29, 2024

Description

Additional context

Issue reference

@github-actions github-actions bot added the BC Breaking change label Apr 29, 2024
Copy link

netlify bot commented Apr 29, 2024

Deploy Preview for spirit-design-system ready!

Name Link
🔨 Latest commit 49f32c5
🔍 Latest deploy log https://app.netlify.com/sites/spirit-design-system/deploys/66333a27609fa20008980bb6
😎 Deploy Preview https://deploy-preview-1382--spirit-design-system.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 96 (no change from production)
Accessibility: 93 (no change from production)
Best Practices: 100 (no change from production)
SEO: 82 (no change from production)
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

@coveralls
Copy link

coveralls commented Apr 29, 2024

Coverage Status

coverage: 80.734% (-15.6%) from 96.371%
when pulling 5318486 on bc/ds-1230-remove-class-for-bordered-alert
into ba9f8d7 on integration/release-v2.

Copy link
Member

@crishpeen crishpeen left a 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.

@curdaj curdaj force-pushed the bc/ds-1230-remove-class-for-bordered-alert branch 2 times, most recently from ce4767f to 1781b68 Compare April 29, 2024 14:29
@curdaj curdaj marked this pull request as ready for review April 30, 2024 09:03
@curdaj curdaj requested review from literat, adamkudrna, pavelklibani and a team as code owners April 30, 2024 09:03
@curdaj curdaj added the run-visual-tests Runs visual regression testing on this PR label Apr 30, 2024
Copy link
Member

@crishpeen crishpeen left a comment

Choose a reason for hiding this comment

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

Good job! :) 💪

Copy link
Contributor

@adamkudrna adamkudrna left a comment

Choose a reason for hiding this comment

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

👍🏻

@curdaj curdaj force-pushed the bc/ds-1230-remove-class-for-bordered-alert branch from 3b50f64 to 5318486 Compare May 2, 2024 06:51
Copy link
Contributor

github-actions bot commented May 2, 2024

@curdaj curdaj force-pushed the bc/ds-1230-remove-class-for-bordered-alert branch from 5318486 to 49f32c5 Compare May 2, 2024 07:00
@curdaj curdaj merged commit 6e223c7 into integration/release-v2 May 2, 2024
27 checks passed
@curdaj curdaj deleted the bc/ds-1230-remove-class-for-bordered-alert branch May 2, 2024 07:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BC Breaking change run-visual-tests Runs visual regression testing on this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants