-
Notifications
You must be signed in to change notification settings - Fork 45
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
DS-590 Notifications component #2392
Conversation
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.
@mikemai2awesome the static demos are looking 💅
I left a few comments, only one is a blocker to merging: #2392 (comment)
packages/components/bolt-notifications/notifications-list-item.schema.js
Show resolved
Hide resolved
Approved, updated tests. I will merge once this passes. |
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.
This all checks out, thanks @mikemai2awesome .
Just an aside, it is hard to look at this schema and know how things are supposed to be assembled, i.e. what belongs inside what. The only way is to look at the examples.
I don't have any brilliant solutions, just something to think about.
Jira
https://pegadigitalit.atlassian.net/browse/DS-590
Summary
Adds a Notifications component to display user notifications.
Details
on
prop to follow conventions. Boolean props default tofalse
.How to test
Run the branch locally and check the Notifications docs. Make sure all props work as expected.