-
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
feat(icons): Watson discovery icons #4696
feat(icons): Watson discovery icons #4696
Conversation
adds: improve-relevance.svg phrase-sentiment.svg folders.svg exit.svg document-sentiment.svg character-patterns.svg updates categories.yml and metadata.yml
cc @conradennis |
Deploy preview for carbon-components-react ready! Built with commit 2c3feba https://deploy-preview-4696--carbon-components-react.netlify.com |
Deploy preview for carbon-elements ready! Built with commit 2c3feba |
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.
Are we trying to use appropriate usage data or still sticking with "This is a description for usage"
Deploy preview for the-carbon-components ready! Built with commit 2c3feba https://deploy-preview-4696--the-carbon-components.netlify.com |
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 once Vince's comment is addressed regarding alias.
removes unpopulated 'aliases' prop
I don't think we've populated that anywhere yet. future property |
`aliases:` a required prop?
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.
looks good !
Changelog
New
32/improve-relevance.svg
32/phrase-sentiment.svg
32/folders.svg
32/exit.svg
32/document-sentiment.svg
32/character-patterns.svg
Changed
Testing / Reviewing
{{ Add descriptions, steps or a checklist for how reviewers can verify this PR works or not }}