-
Notifications
You must be signed in to change notification settings - Fork 73
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
(OUI Docs) Notification event #88
Comments
Evaluate Notification event in its current state from the above list - Is it auditing task for notification event? |
@kaddy645 will look into this. |
@KrooshalUX I do not think we are using this component anywhere in OSD. looking at its current state it looks good to use. In future we might need to update some of its prop if needed. |
@kaddy645 correct, we are not using it yet. I mostly just wanted to know if there is anything you might find that would prevent us from using it - sounds like no concern with its current implementation and documentation. Thanks! |
@kaddy645 did you notice any differences between the component and the documentation? if not, we can check these tasks off and close the issue, and i can remove beta from the docs pages (tracked in the meta docs issue) |
Component looks good to me. there are few required props we have in docs that end user will need when using this component. there are number of other props, but as long they are not required we are good. (just for e.g )
|
Updated description relevant to docs site task |
The text was updated successfully, but these errors were encountered: