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

chore!: Support React 18 [WEB-2055] #2643

Merged
merged 69 commits into from
Jan 19, 2023
Merged

chore!: Support React 18 [WEB-2055] #2643

merged 69 commits into from
Jan 19, 2023

Conversation

jakemhiller
Copy link
Member

@jakemhiller jakemhiller commented Jan 6, 2023

Overview

This PR adds support for React 18 WITHOUT actually using React 18 locally for tests or storybook. We will be updating those to React 18 as a follow up. This should allow clients to update to React 18.

Add support for React 18
This is a breaking change for gamut-styles because we have removed react-helmet from the AssetProvider component. You'll need to wrap AssetProvider in an instance of react-helmet in your app.

PR Checklist

  • Related to designs:
  • Related to JIRA ticket:WEB-2055
  • I have run this code to verify it works
  • This PR includes unit tests for the code change
  • This PR includes testing instructions tests for the code change
  • The alpha package of this PR is passing end-to-end tests in all relevant Codecademy repositories

Testing Instructions

Don't make me tap the sign.

PR Links and Envs

Repository PR Link PR Env
Monolith Monolith PR Monolith Env
Portal Portal PR Portal Env

dreamwasp and others added 23 commits October 3, 2022 12:52
* origin/main: (64 commits)
  chore(release): publish
  feat(Toaster): Use current color mode in `Toaster`
  chore(release): publish
  chore: update tracking package
  chore(release): publish
  fix(RatingsBar): fixes the focus on the wrapper
  chore(release): publish
  refactor(ScoreSummary): add optional description text
  chore(release): publish
  chore(Gamut + setupEnzyme): deprecate enzyme + remove tests
  chore(release): publish
  fix(ListSection): Filter out falsy children
  chore(release): publish
  fix(SelectDropdown): fix remove + remove all  buttons + medium sizing change bug for `multiple` variant
  chore(release): publish
  chore: change enzyme tests to rtl in gamut-labs
  chore(release): publish
  fix(RatingsBar): Adds stroke between yellow bar and white background
  chore(release): publish
  chore(deps): bump tracking from 0.26.0 to 1.0.1
  ...
* origin/main:
  chore(release): publish
  chore: update react imports and tsconfig for jsx runtime
  chore(release): publish
  feat(LayoutMenuVariant): create layout menu variant for experiment #2199
@nx-cloud
Copy link

nx-cloud bot commented Jan 6, 2023

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 9461f1c. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this branch


✅ Successfully ran 4 targets

Sent with 💌 from NxCloud.

* main: (30 commits)
  chore(release): publish
  feat(FormGroupLabel, GridForm): assorted embedded user-form features
  chore(release): publish
  fix(Flyout): move background color to fix overflow
  chore(release): publish
  chore: update local node version to v16 [WEB-2205]
  chore(release): publish
  refactor(ScoreSummary): make ScoreSummary responsive
  chore(release): publish
  feat(Flyout): Added optional background prop (#2627)
  chore(release): publish
  fix: update width of page alert to match page contents
  chore(release): publish
  chore(AppHeader)!: Remove bookmarks and favorites
  chore(release): publish
  fix(RatingsBar): Conditional box shadow (hide if 0%)
  chore(release): publish
  refactor(ScoreSummary): make trackSlug and tracking optional in ScoreSummary
  chore(release): publish
  feat(Footer): rename Pro Membership link
  ...
@jakemhiller jakemhiller changed the title React 18 chore: Support React 18 Jan 12, 2023
@jakemhiller jakemhiller changed the title chore: Support React 18 chore: Support React 18 [WEB-2055] Jan 12, 2023
@jakemhiller jakemhiller changed the title chore: Support React 18 [WEB-2055] chore!: Support React 18 [WEB-2055] Jan 17, 2023
@jakemhiller jakemhiller marked this pull request as ready for review January 17, 2023 20:49
@jakemhiller jakemhiller requested a review from a team as a code owner January 17, 2023 20:49
@codecademydev
Copy link
Collaborator

🚀 Styleguide deploy preview ready!

https://63c84d4bc69c410a4c055284--gamut-preview.netlify.app

Deploy Logs

@codecademydev
Copy link
Collaborator

📬Published Alpha Packages:

@codecademy/gamut-icons@7.9.8-alpha.9461f1.0
@codecademy/gamut-illustrations@0.32.10-alpha.9461f1.0
@codecademy/gamut-kit@0.6.266-alpha.9461f1.0
@codecademy/gamut-labs@34.1.1-alpha.9461f1.0
@codecademy/gamut-patterns@0.8.23-alpha.9461f1.0
@codecademy/gamut-styles@14.2.6-alpha.9461f1.0
@codecademy/gamut-tests@4.0.23-alpha.9461f1.0
@codecademy/gamut@51.7.4-alpha.9461f1.0
@codecademy/markdown-overrides@0.9.20-alpha.9461f1.0
@codecademy/monaco@0.6.121-alpha.9461f1.0
@codecademy/styleguide@62.8.7-alpha.9461f1.0
@codecademy/variance@0.20.6-alpha.9461f1.0

Copy link
Contributor

@aresnik11 aresnik11 left a comment

Choose a reason for hiding this comment

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

Changes make sense and confirmed monolith and portal look good in their corresponding PRs!

@jakemhiller jakemhiller added the Ship It :shipit: Automerge this PR when possible label Jan 19, 2023
@codecademydev codecademydev merged commit 89d6b22 into main Jan 19, 2023
@codecademydev codecademydev deleted the jh-cass-web-2055-2 branch January 19, 2023 15:38
@codecademydev codecademydev removed the Ship It :shipit: Automerge this PR when possible label Jan 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants