Skip to content
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

fix(react): update downshift to 5.2.1 #6418

Merged
merged 3 commits into from
Jul 13, 2020

Conversation

joshblack
Copy link
Contributor

Closes #6383

Updates downshift to 5.2.1 to cover a regression when updating to 5.x that caused setState to be called during render.

Changelog

New

Changed

  • Update downshift to 5.2.1

Removed

Testing & Reviewing

  • Download the codesandbox used for the bug report here: https://codesandbox.io/s/sad-pond-pgwrv
  • Checkout this PR locally in the monorepo and run yarn install
  • Update the codesandbox you downloaded to point carbon-components-react to the package in your monorepo. For example, mine looks like:
{
  "dependencies": {
    "carbon-components": "10.16.0-rc.0",
    "carbon-components-react": "file:/Users/joshblack/work/carbon/monorepo/packages/react",
    "carbon-icons": "latest",
    "react": "latest",
    "react-dom": "latest"
  }
}

@joshblack joshblack requested a review from a team as a code owner July 8, 2020 22:13
@ghost ghost requested review from aledavila and dakahn July 8, 2020 22:13
@netlify
Copy link

netlify bot commented Jul 8, 2020

Deploy preview for carbon-elements ready!

Built with commit 83d3ef8

https://deploy-preview-6418--carbon-elements.netlify.app

@netlify
Copy link

netlify bot commented Jul 8, 2020

Deploy preview for carbon-components-react ready!

Built with commit 83d3ef8

https://deploy-preview-6418--carbon-components-react.netlify.app

@joshblack
Copy link
Contributor Author

@dakahn @aledavila let me know if you have any questions!

Copy link
Collaborator

@tw15egan tw15egan left a comment

Choose a reason for hiding this comment

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

🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Dropdown] React warning: Cannot update a component while rendering a different component (Dropdown)
3 participants