-
Notifications
You must be signed in to change notification settings - Fork 130
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
feat(components): migrate Notification components to TypeScript #637
Conversation
BREAKING CHANGE: The **InlineMessage** component's `type` prop has been renamed to `variant` (🤖 _inline-message-variant-enum_) feature/typescript-messages
feature/typescript-messages
BREAKING CHANGE: The NotificationIcon and NotificationButton components have been removed. Use the Notification component's icon and children props instead. feature/typescript-messages
feature/typescript-messages
feature/typescript-messages
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/sumup-oss/circuit-ui/k27ltdozp |
Codecov Report
@@ Coverage Diff @@
## beta #637 +/- ##
==========================================
- Coverage 93.71% 93.69% -0.03%
==========================================
Files 133 131 -2
Lines 2213 2219 +6
Branches 612 617 +5
==========================================
+ Hits 2074 2079 +5
- Misses 117 118 +1
Partials 22 22
|
🎉 This PR is included in version 2.0.0-beta.17 🎉 The release is available on: Your semantic-release bot 📦🚀 |
🎉 This PR is included in version 2.0.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Addresses #516.
Purpose
The current Notification component composition is quite brittle because it depends on the proper positioning of each component to work. It also makes it impossible to change the layout and composition of the notifications in the future. This is solved by replacing the subcomponents (NotificationIcon and NotificationButton) with props on the main Notification component.
Approach and changes
BREAKING CHANGE:
The NotificationIcon and NotificationButton components have been removed. Use the Notification component's
icon
andchildren
props instead.Definition of done