Add info type to intent

segmentio / evergreen

🌲 Evergreen React UI Framework by Segment

https://evergreen.segment.com

MIT License

12.39k stars 832 forks source link

Add info type to intent #1331

Closed amglt closed 3 years ago

amglt commented 3 years ago

Overview

Fix for the issue #1319

I realized that "info" is exactly the same as "none" but doesn't exist in the documentation. This PR add "info" to intent proptype to avoid having warnings but I could also remove the type from TypeScript definition.

Screenshots (if applicable)

Documentation

[x] Updated Typescript types and/or component PropTypes
[ ] Added / modified component docs
[x] Added / modified Storybook stories

netlify[bot] commented 3 years ago

✔️ Deploy Preview for evergreen-storybook ready!

🔨 Explore the source changes: ecfd2b5caa0b8dae199879803938998bb5018f88

🔍 Inspect the deploy log: https://app.netlify.com/sites/evergreen-storybook/deploys/61546481de19980008922bcb

😎 Browse the preview: https://deploy-preview-1331--evergreen-storybook.netlify.app

brandongregoryscott commented 3 years ago

Thanks for the contribution! Since we loosened the typing restrictions in #1341, this is probably not needed.