-
Notifications
You must be signed in to change notification settings - Fork 799
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
Updated notification/usage.mdx with content-focused edits #4434
base: main
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Hi @alisonjoseph, @guidari, could you please help with the deploy preview for this PR, thanks! |
This reverts commit 14bb32e.
@laurenmrice Hi, Lauren. Can you indicate the status of the PR related to this issue, please? It seems like there's movement, but the changes haven't been implemented yet. |
@lesnider You can view your changes by clicking on this link in this PR. Looks like your changes have been made there. |
@laurenmrice Thanks, Lauren. Do you know when they will go live? I am not seeing them in the guidelines yet. |
@lesnider Once this PR merges the changes will go live on the real website. We need to get an approved review first on this PR by a Carbon developer and a Carbon designer before we can merge it. I have put this on my next sprint (starting Thursday), to give this a review from a design side. |
Related to #4165
Content edits to Notification component topic