Skip to content

chore: upgrade next to 15.2.0-canary.1#1874

Merged
fikrikarim merged 1 commit intosoul/mainfrom
fikri/eng-7189-follow-up-a-bug-in-nextjs-cannot-revalidate-404-page-with
Jan 8, 2025
Merged

chore: upgrade next to 15.2.0-canary.1#1874
fikrikarim merged 1 commit intosoul/mainfrom
fikri/eng-7189-follow-up-a-bug-in-nextjs-cannot-revalidate-404-page-with

Conversation

@fikrikarim
Copy link
Copy Markdown
Member

What/Why?

This release contains fixes for the 404 revalidation issue with revalidateTag. vercel/next.js#74577

One quirk about the issue is that this issue only appears when Next.js returns 404 as the HTTP status. Sometimes PPR will return 200 HTTP status even though we throw notFound inside a page. Related docs.

To be able to reproduce this issue locally, I need to revert the temporary cookies() error commit. Otherwise, the not found page will have 200 as the HTTP status, and won't have this issue in the first place.

Testing

Before

revalidation-before.mp4

After

revalidation-after.mp4

This release contain fixes for 404 revalidation issue with revalidateTag bug.
@fikrikarim fikrikarim requested review from a team and migueloller January 8, 2025 07:34
@linear
Copy link
Copy Markdown

linear Bot commented Jan 8, 2025

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Jan 8, 2025

⚠️ No Changeset found

Latest commit: 3227f09

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@vercel
Copy link
Copy Markdown

vercel Bot commented Jan 8, 2025

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

Name Status Preview Comments Updated (UTC)
catalyst-latest ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 8, 2025 7:34am
catalyst-soul ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 8, 2025 7:34am
3 Skipped Deployments
Name Status Preview Comments Updated (UTC)
catalyst ⬜️ Ignored (Inspect) Jan 8, 2025 7:34am
catalyst-au ⬜️ Ignored (Inspect) Jan 8, 2025 7:34am
catalyst-uk ⬜️ Ignored (Inspect) Jan 8, 2025 7:34am

@fikrikarim fikrikarim merged commit 53b3215 into soul/main Jan 8, 2025
@fikrikarim fikrikarim deleted the fikri/eng-7189-follow-up-a-bug-in-nextjs-cannot-revalidate-404-page-with branch January 8, 2025 13:23
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