Skip to content

Conversation

@wuweiweiwu
Copy link
Contributor

selected prop is missing in the TypeScript definition but defined in PropTypes

Closes # (type the issue number after # if applicable; otherwise remove this line)

Screenshots

Please provide before/after screenshots for any visual changes

Merge checklist

  • Added or updated TypeScript definitions (index.d.ts) if necessary
  • Added/updated tests
  • Added/updated documentation
  • Tested in Chrome
  • Tested in Firefox
  • Tested in Safari
  • Tested in Edge

Take a look at the What we look for in reviews section of the contributing guidelines for more information on how we review PRs.

@vercel
Copy link

vercel bot commented Mar 27, 2020

This pull request is being automatically deployed with ZEIT Now (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://zeit.co/primer/primer-components/ppmrzxzwl
✅ Preview: https://primer-components-git-fork-wuweiweiwu-hwu-fix-sidenav-types.primer.now.sh

@vercel vercel bot temporarily deployed to Preview March 27, 2020 23:02 Inactive
@wuweiweiwu wuweiweiwu changed the title fix: SideNav.Link types is missing selected fix: SideNav.Link types is missing selected Mar 27, 2020
@emplums emplums added the minor release new features label Mar 30, 2020
Copy link

@emplums emplums left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @wuweiweiwu! I'll merge this into the minor tracking branch once it's up 🙌

@emplums emplums merged commit f08e6a1 into primer:master Mar 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

minor release new features

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants