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

[NEW] Migrating to NextJS 13 #192

Merged
merged 8 commits into from
Nov 5, 2022
Merged

Conversation

Dnouv
Copy link
Member

@Dnouv Dnouv commented Oct 31, 2022

Changes in the PR:

  • Remove the unused jitsi/web-sdk
  • Remove dependency on rc-fuselage
  • update react-slick version
  • updated ReactJS to v18
  • added an override for next-auth conflict for the time being until the dependency conflict fix is published by next-auth parent repo
  • Updated NextJS to v13

Thank you!

@lgtm-com
Copy link

lgtm-com bot commented Nov 1, 2022

This pull request introduces 3 alerts when merging 60a3d92 into 7241ded - view on LGTM.com

new alerts:

  • 3 for Useless conditional

@lgtm-com
Copy link

lgtm-com bot commented Nov 4, 2022

This pull request introduces 3 alerts when merging 7461d79 into 9dd8006 - view on LGTM.com

new alerts:

  • 3 for Useless conditional

@lgtm-com
Copy link

lgtm-com bot commented Nov 4, 2022

This pull request introduces 3 alerts when merging 2eddd56 into 9dd8006 - view on LGTM.com

new alerts:

  • 3 for Useless conditional

@lgtm-com
Copy link

lgtm-com bot commented Nov 4, 2022

This pull request introduces 3 alerts when merging 4ab8b21 into 9dd8006 - view on LGTM.com

new alerts:

  • 3 for Useless conditional

@Dnouv Dnouv marked this pull request as ready for review November 4, 2022 18:33
@Sing-Li Sing-Li changed the title [WIP][NEW] Migrating to NextJS 13 [NEW] Migrating to NextJS 13 Nov 5, 2022
@Sing-Li Sing-Li merged commit 73631c8 into RocketChat:master Nov 5, 2022
@Sing-Li
Copy link
Member

Sing-Li commented Nov 5, 2022

close #190

@Sing-Li
Copy link
Member

Sing-Li commented Nov 5, 2022

closes #190

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.

2 participants