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

Bump react and react-dom #10866

Merged
merged 1 commit into from
Mar 30, 2022

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Mar 29, 2022

Bumps react and react-dom. These dependencies needed to be updated together.
Updates react from 17.0.2 to 18.0.0

Commits

Updates react-dom from 17.0.2 to 18.0.0

Commits

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Bumps [react](https://github.com/facebook/react/tree/HEAD/packages/react) and [react-dom](https://github.com/facebook/react/tree/HEAD/packages/react-dom). These dependencies needed to be updated together.

Updates `react` from 17.0.2 to 18.0.0
- [Release notes](https://github.com/facebook/react/releases)
- [Changelog](https://github.com/facebook/react/blob/main/CHANGELOG.md)
- [Commits](https://github.com/facebook/react/commits/HEAD/packages/react)

Updates `react-dom` from 17.0.2 to 18.0.0
- [Release notes](https://github.com/facebook/react/releases)
- [Changelog](https://github.com/facebook/react/blob/main/CHANGELOG.md)
- [Commits](https://github.com/facebook/react/commits/HEAD/packages/react-dom)

---
updated-dependencies:
- dependency-name: react
  dependency-type: direct:production
  update-type: version-update:semver-major
- dependency-name: react-dom
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
@dependabot dependabot bot added dependencies Pull requests that update a dependency file JavaScript labels Mar 29, 2022
@gitpod-io
Copy link

gitpod-io bot commented Mar 29, 2022

@codeclimate
Copy link

codeclimate bot commented Mar 29, 2022

Code Climate has analyzed commit 374ae1b and detected 0 issues on this pull request.

View more on Code Climate.

@github-actions
Copy link

This pull request generated screenshots of many common pages in the running app. You should be able to download and view them here:
https://github.com/publiclab/plots2/suites/5847456203/artifacts/196731288

@jywarren
Copy link
Member

This should be well tested, but we may need to visually confirm manually following guide in https://github.com/publiclab/plots2/blob/3fba3036d155032031683996661043bf839c8ab8/app/javascript/components/REACT_COMMENTS.md#how-to-access

visit any research note and copy-and-paste the following parameter to the end of the URL:

?react=true

Then try commenting.

@PeculiarE
Copy link
Contributor

@jywarren, I ran the PR on Gitpod and visually confirmed that the react commenting system works just fine. Please see below the video recording to this effect. Thank you!

react-comment-demo.1.1.mp4

@jywarren
Copy link
Member

Hi @PeculiarE thank you so much. I'm a little confused however i don't quite get why GitPod isn't showing you the full URL of the page you're on. It should look something like https://publiclab.org/notes/junior_walk1337/03-28-2022/hazy-3-28-22, not just ending in /notes, you know? But in your screencast (which is fantastic by the way) i see just /notes?react=true:

image

Could you try clicking this in the upper right to open it in a new browser window and hopefully we get the true complete URL?

image

Does that make sense? Thank you and sorry to ask you to go through so much more trouble!

@PeculiarE
Copy link
Contributor

Yes, that makes sense @jywarren. I just assumed the browser within the GitPod environment was showing me the full URL, but it looks like that's not the case. Would check and revert shortly. Thank you.

@PeculiarE
Copy link
Contributor

So yes 😄 , we are able to get the true complete URL by clicking on the new-browser-window icon. And this is it:

https://3000-publiclab-plots2-1097ad9pz5m.ws-eu38.gitpod.io/notes/admin/03-30-2022/blog-post?react=true

Also made another screencast for a more detailed demo. Thank you!

react-comment-demo.1.2.mp4

@jywarren
Copy link
Member

Super. Thank you so much!!!

@jywarren jywarren merged commit 121ff0e into main Mar 30, 2022
@dependabot dependabot bot deleted the dependabot/npm_and_yarn/react-and-react-dom-18.0.0 branch March 30, 2022 18:59
@jywarren
Copy link
Member

You also got a 👍 from @noi5e who designed and built this system! 🎉 and just also merged a related update at #10867. You can read more at #9365 !

@PeculiarE
Copy link
Contributor

Thank you! @jywarren 🥰

And yeah, the thumbs up is a huge compliment from @noi5e (thank you!!!! @noi5e ) 😄 😄 ....Would definitely check out both the PR and the thread. Thanks once again ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file JavaScript
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants