Skip to content

Upgrade react to version 17.0.0 #15

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

Closed
wants to merge 1 commit into from
Closed

Conversation

depfu[bot]
Copy link
Contributor

@depfu depfu bot commented Oct 21, 2020

Here is everything you need to know about this upgrade. Please take a good look at what changed and the test results before merging this pull request.

What changed?

✳️ react (^16.13.1 → ^17.0.0) · Repo · Changelog

Release Notes

17.0.0

Today, we are releasing React 17!

Learn more about React 17 and how to update to it on the official React blog.

React

React DOM

  • Delegate events to roots instead of document. (@trueadm in #18195 and others)
  • Clean up all effects before running any next effects. (@bvaughn in #17947)
  • Run useEffect cleanup functions asynchronously. (@bvaughn in #17925)
  • Use browser focusin and focusout for onFocus and onBlur. (@trueadm in #19186)
  • Make all Capture events use the browser capture phase. (@trueadm in #19221)
  • Don't emulate bubbling of the onScroll event. (@gaearon in #19464)
  • Throw if forwardRef or memo component returns undefined. (@gaearon in #19550)
  • Remove event pooling. (@trueadm in #18969)
  • Stop exposing internals that won’t be needed by React Native Web. (@necolas in #18483)
  • Attach all known event listeners when the root mounts. (@gaearon in #19659)
  • Disable console in the second render pass of DEV mode double render. (@sebmarkbage in #18547)
  • Deprecate the undocumented and misleading ReactTestUtils.SimulateNative API. (@gaearon in #13407)
  • Rename private field names used in the internals. (@gaearon in #18377)
  • Don't call User Timing API in development. (@gaearon in #18417)
  • Disable console during the repeated render in Strict Mode. (@sebmarkbage in #18547)
  • In Strict Mode, double-render components without Hooks too. (@eps1lon in #18430)
  • Allow calling ReactDOM.flushSync during lifecycle methods (but warn). (@sebmarkbage in #18759)
  • Add the code property to the keyboard event objects. (@bl00mber in #18287)
  • Add the disableRemotePlayback property for video elements. (@tombrowndev in #18619)
  • Add the enterKeyHint property for input elements. (@eps1lon in #18634)
  • Warn when no value is provided to <Context.Provider>. (@charlie1404 in #19054)
  • Warn when memo or forwardRef components return undefined. (@bvaughn in #19550)
  • Improve the error message for invalid updates. (@JoviDeCroock in #18316)
  • Exclude forwardRef and memo from stack frames. (@sebmarkbage in #18559)
  • Improve the error message when switching between controlled and uncontrolled inputs. (@vcarl in #17070)
  • Keep onTouchStart, onTouchMove, and onWheel passive. (@gaearon in #19654)
  • Fix setState hanging in development inside a closed iframe. (@gaearon in #19220)
  • Fix rendering bailout for lazy components with defaultProps. (@jddxf in #18539)
  • Fix a false positive warning when dangerouslySetInnerHTML is undefined. (@eps1lon in #18676)
  • Fix Test Utils with non-standard require implementation. (@just-boris in #18632)
  • Fix onBeforeInput reporting an incorrect event.type. (@eps1lon in #19561)
  • Fix event.relatedTarget reported as undefined in Firefox. (@claytercek in #19607)
  • Fix "unspecified error" in IE11. (@hemakshis in #19664)
  • Fix rendering into a shadow root. (@Jack-Works in #15894)
  • Fix movementX/Y polyfill with capture events. (@gaearon in #19672)
  • Use delegation for onSubmit and onReset events. (@gaearon in #19333)
  • Improve memory usage. (@trueadm in #18970)

React DOM Server

  • Make useCallback behavior consistent with useMemo for the server renderer. (@alexmckenley in #18783)
  • Fix state leaking when a function component throws. (@pmaccart in #19212)

React Test Renderer

Concurrent Mode (Experimental)

Artifacts

Does any of this look wrong? Please let us know.

Commits

See the full diff on Github. The new version differs by more commits than we can show here.


Depfu Status

Depfu will automatically keep this PR conflict-free, as long as you don't add any commits to this branch yourself. You can also trigger a rebase manually by commenting with @depfu rebase.

All Depfu comment commands
@​depfu rebase
Rebases against your default branch and redoes this update
@​depfu recreate
Recreates this PR, overwriting any edits that you've made to it
@​depfu merge
Merges this PR once your tests are passing and conflicts are resolved
@​depfu close
Closes this PR and deletes the branch
@​depfu reopen
Restores the branch and reopens this PR (if it's closed)
@​depfu pause
Ignores all future updates for this dependency and closes this PR
@​depfu pause [minor|major]
Ignores all future minor/major updates for this dependency and closes this PR
@​depfu resume
Future versions of this dependency will create PRs again (leaves this PR as is)

@depfu depfu bot added the depfu label Oct 21, 2020
@depfu depfu bot requested a review from yaroslav-ilin October 21, 2020 21:35
@codecov
Copy link

codecov bot commented Oct 21, 2020

Codecov Report

Merging #15 into main will decrease coverage by 22.73%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##             main      #15       +/-   ##
===========================================
- Coverage   68.76%   46.02%   -22.74%     
===========================================
  Files          41       41               
  Lines         365      365               
  Branches       80       80               
===========================================
- Hits          251      168       -83     
- Misses        109      195       +86     
+ Partials        5        2        -3     
Impacted Files Coverage Δ
src/lib/emoji.ts 0.00% <0.00%> (-100.00%) ⬇️
src/view/components/traceroute/_country_item.tsx 4.54% <0.00%> (-86.37%) ⬇️
src/view/components/traceroute/cloudflare.tsx 16.66% <0.00%> (-83.34%) ⬇️
src/view/components/traceroute/index.tsx 12.50% <0.00%> (-81.25%) ⬇️
src/view/components/traceroute/ip.tsx 13.04% <0.00%> (-78.27%) ⬇️
src/view/components/sequence.tsx 28.57% <0.00%> (-71.43%) ⬇️
src/view/components/typography.tsx 37.50% <0.00%> (-62.50%) ⬇️
src/view/components/header_primary.tsx 12.50% <0.00%> (-62.50%) ⬇️
src/view/components/popup_content.tsx 33.33% <0.00%> (-50.01%) ⬇️
src/view/components/logo.tsx 25.00% <0.00%> (-50.00%) ⬇️
... and 1 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2537b78...2f0e328. Read the comment docs.

@depfu
Copy link
Contributor Author

depfu bot commented Oct 23, 2020

Closed in favor of #18.

@depfu depfu bot closed this Oct 23, 2020
@depfu
Copy link
Contributor Author

depfu bot commented Oct 23, 2020

Closed in favor of #18.

@depfu depfu bot deleted the depfu/update/npm/react-17.0.0 branch October 23, 2020 13:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

0 participants