-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
docs(Slug): add dev docs to Slug overview #15812
Conversation
✅ Deploy Preview for v11-carbon-react ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@tw15egan this is excellent. What about the other components? Are there docs explaining how to pass in a Slug component into the slug prop? |
@mattrosno I can add that. Would it make sense to keep it all in this overview, or should I add it to the examples folder? I’ll add some DataTable docs as well |
@tw15egan at minimum, maybe something in all 4 Overviews, even if 3 of them just point you to the first that includes all documentation. No strong preference here besides having something mentioned in all Overviews. |
@mattrosno updated |
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.
LGTM! 🚀
@mattrosno fixed 👍🏻 |
50c634e
Hey there! v11.52.0 was just released that references this issue/PR. |
Closes #15784
Adds in documentation regarding the usage of the
Slug
component and its propsChangelog
New
Slug
component and a few of its propsChanged
Testing / Reviewing
Ensure the spelling/grammar is correct and the content makes sense. Is there anything else we are missing?