Skip to content

Wech 21/support react 18 #311

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 65 commits into from
Closed

Wech 21/support react 18 #311

wants to merge 65 commits into from

Conversation

martimalek
Copy link
Contributor

@martimalek martimalek commented Mar 17, 2023

What:

Migrate to react 18


Why:

We want to stop supporting react<18 in Wave 2.0 (#239)

How:

  • Bump react and react-dom versions to ^18.2.0 (not sure if we should target ^18.0.0 instead)
  • Use react's useId instead of the internal generateId in useGeneratedId

Checklist:

  • Fix type errors
  • Use react's useId in Month.tsx instead of the internal generateId
  • Remove our internal util generateId once it's no longer used
  • Remove nanoid dependency
  • Remove IE11 polyfill required for nanoid once it's no longer used (basically remove polyfills.ts)
  • Fix dependency errors (docz is not compatible with React 18)
  • Ready to be merged

Closes #239

@martimalek martimalek added the breaking change Breaking changes that require a major version release label Mar 17, 2023
@martimalek martimalek added this to the Wave@2 milestone Mar 17, 2023
@martimalek martimalek self-assigned this Mar 17, 2023
@martimalek martimalek mentioned this pull request Mar 17, 2023
7 tasks
zackjones11 and others added 3 commits March 20, 2023 12:34
Docz is not supported anymore. We decided to migrate to a stable solution

Co-authored-by: Zack Jones <[email protected]>
Co-authored-by: Nikolai Lopin <[email protected]>
Components migrated: 
* Accordion
* Banner
* Box
* Card
* Checkbox
* Datepicker
* Dimmer
* Divider
* Filepicker
---------

Co-authored-by: Zack Jones <[email protected]>
Co-authored-by: Cristian Sanchez <[email protected]>
Co-authored-by: Nikolai Lopin <[email protected]>
@martimalek martimalek mentioned this pull request Apr 6, 2023
1 task
JanHamara and others added 10 commits April 14, 2023 11:15
* docs: add input story

* docs: add label story

* docs: add link story

* docs: add modal story

* docs: add offcanvas story

* docs: define argTypes for Input and Label stories

* docs(password): add Password to Storybook

* docs(phoneinput): add PhoneInput to Storybook

* fix(test): update snapshots

* docs(pagination): add Pagination to Storybook

* docs(radiobutton): add RadioButton to Storybook

* docs(popover): add Popover to Storybook

---------

Co-authored-by: Aleksandar Spasov <[email protected]>
Co-authored-by: Ján Hamara <[email protected]>
Update typescript to the lowest supported version
* Input
* Label
* Link
* Modal
* Offcanvas

Co-authored-by: Aleksandar Spasov <[email protected]>
Co-authored-by: Nikolai Lopin <[email protected]>
* table
* tag
* text
* textarea
* toggle
* tooltip
---------

Co-authored-by: Christian <[email protected]>
Co-authored-by: Nikolai Lopin <[email protected]>
* Search
* TabBar
* Select
* SelectList
* Skeleton

Co-authored-by: Inga Schünemann <[email protected]>
* essentials pages
* Logo
* Icons
* remove IE 11 from the list of supported

---------

Co-authored-by: lena <[email protected]>
Co-authored-by: Vanessa Knibbe <[email protected]>
Co-authored-by: Nikolai Lopin <[email protected]>
@martimalek martimalek force-pushed the WECH-21/support-react-18 branch from d95ae7a to a045eb8 Compare April 21, 2023 08:38
JanHamara and others added 3 commits May 25, 2023 15:10
* feat(landing): add landing page for Storybook

* docs(landing): finish the landing page integration

* docs(theme): theme storybook with our branding

* refactor(landingpage): mr review modifs part 1

* refactor(landing): wrap landing page mdx in unstyled component

---------

Co-authored-by: Ján Hamara <[email protected]>
Fix up default font for theme to Open Sans (my bad, I left behind one commit and merged in the MR before pushing it)

Relevant MR: #334
Update 'by Freenow' SVG with one with correct font (my bad, I left behind one commit and merged in the MR before pushing it)

Relevant MR: #334
@nlopin
Copy link
Contributor

nlopin commented May 26, 2023

I merged the branch to the next manually. Closing the PR

@nlopin nlopin closed this May 26, 2023
@martimalek
Copy link
Contributor Author

@nlopin Why did you decide to do it manually instead of merging this?

@nlopin nlopin deleted the WECH-21/support-react-18 branch October 17, 2023 13:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking change Breaking changes that require a major version release
Development

Successfully merging this pull request may close these issues.

9 participants