Skip to content

[HUD] upgrade to nextjs 13#5862

Closed
yangw-dev wants to merge 18 commits intomainfrom
upgradeNextJs13
Closed

[HUD] upgrade to nextjs 13#5862
yangw-dev wants to merge 18 commits intomainfrom
upgradeNextJs13

Conversation

@yangw-dev
Copy link
Copy Markdown
Contributor

@yangw-dev yangw-dev commented Nov 5, 2024

@vercel
Copy link
Copy Markdown

vercel bot commented Nov 5, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
torchci ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 6, 2024 6:57pm

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Nov 5, 2024
@clee2000 clee2000 mentioned this pull request Nov 6, 2024
@yangw-dev yangw-dev requested a review from clee2000 November 7, 2024 02:05
clee2000 added a commit that referenced this pull request Nov 8, 2024
Upgrading cuz I think it's a good idea to be on a more recent version.
idk what the consequences will be

Theres also next 15.something but that came out really recently and
next-auth doesn't list it as a compatible version

octokit types changed which is why webhookToDyanamo looks different.
Functionality should be the same, mostly just type changes + casting,
and we have tests after #5869

Cousin to #5862 but does both
slightly less and slightly more
@yangw-dev yangw-dev closed this Sep 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants