-
Notifications
You must be signed in to change notification settings - Fork 26.7k
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
On-Demand Revalidation does not revalidate when using the App Router and deployed to Vercel #50714
Comments
I'm experiencing a similar issue, and both |
I have the same issue. When running locally using |
similar issue: #49861 btw, I just opened a Vercel support ticket. because it suddenly doesn't worked for me from June 2. |
Debug info:
This appears when running npm run dev with turbo. It works fine without turbo, though in an npm run build it doesnt work |
Is there any workaround? Is it working in any previous version? |
may this issue be related with this feature being billed? https://vercel.com/docs/infrastructure/data-cache/limits-and-pricing#usage |
another similar issue: #49387 |
@MaxiSantos I don't believe so. Hobby/Free accounts are allowed 100/validations per account according to the docs. |
@marianobenitezz I haven't found a direct workaround to the issue, but for my use case I was able to implement Draft Mode on my staging instance. Of course this is not a great solution for production environments. |
Hi, We have recently pushed couple of ISR related fix in Next.js version 13.4.5. Could you try and let us know how it goes? |
It still doesn't appear to be working, tested with |
I've also tried updating to v13.4.5, but it doesn't seem to be working. |
Thank you! We will investigate this separately and report back! |
here is a link for a reproducible issue https://codesandbox.io/p/sandbox/next-js-fxis37 |
There is also this bug on netlify, so it's not Vercel's fault. The weird thing is that it works on local machines but not on hosting |
I get this on some revalidation attempts and not on others (local and deployed to a private server). I'm not sure what makes them different, but it doesn't seem to be 100% reproducible for me. |
Would route groups have any effect on this? Like if my filesystem path is actually |
In my case, it turned out that I wasn't passing the native |
I'm working with data pulled directly from the database. On a settings page, for example, I have a button that auto-updates some information and saves that to the database. The page looks fine and the data saves, but if they navigate away from that page and click back to it, the page will show the old data. So this is where I thought I could use
|
you can use |
@nabildroid, I'm pulling data directly from my DB via external functions. I'm not using Fetch. Thus, it appears from the documentation that tags won't work for me since they only work on Fetch. |
You can use unstable_cache to make it work, but ya.. |
There's a hacky way to still achieve it, just fetch 65base image |
@nabildroid what do you mean with 65base image? |
Fetch something like this |
Any update on this? It seems |
@palmamartin We will need more specific information for what you're seeing exactly and a reproduction. These functions are working correctly in production for our website, as well as other demo sites and tests. That doesn't mean there might be a bug, but it's hard to say unless there's specific details.
My assumption here is that it doesn't have anything to do with tagging fetches, but if you are trying to generate a large site, are you trying to build many static pages? How long does it take to run |
Please verify that your issue can be recreated with Why was this issue marked with the
|
Hey @leerob, what we noticed is that
Around 111 static pages. Here the times for building it locally:
We currently tag the each individual static page using the following schema: Just retried enabling tags and the build still keeps running until it gets killed by Vercel. The project code is private so I cannot share it publicly unfortunately. But here is the build log of Vercel:
|
I believe that either the documentation is wrong or there's a bug in Next.js. Implicit |
This issue has been automatically closed because it wasn't verified against next@canary. If you think it was closed by accident, please leave a comment. If you are running into a similar issue, please open a new issue with a reproduction. Thank you. |
I used canary 13.4.20-canary.15 and |
This closed issue has been automatically locked because it had no new activity for 2 weeks. If you are running into a similar issue, please create a new issue with the steps to reproduce. Thank you. |
Verify canary release
Provide environment information
Operating System: Platform: darwin Arch: arm64 Version: Darwin Kernel Version 22.4.0: Mon Mar 6 20:59:28 PST 2023; root:xnu-8796.101.5~3/RELEASE_ARM64_T6000 Binaries: Node: 18.15.0 npm: 9.5.0 Yarn: 1.22.19 pnpm: 8.5.1 Relevant packages: next: 13.4.5-canary.4 eslint-config-next: N/A react: 18.2.0 react-dom: 18.2.0 typescript: 4.9.5
Which area(s) of Next.js are affected? (leave empty if unsure)
App directory (appDir: true), Middleware / Edge (API routes, runtime)
Link to the code that reproduces this issue or a replay of the bug
https://github.com/austintreneff/reproduction-app-revalidate
To Reproduce
/api/revalidate
Describe the Bug
When deployed to Vercel, On-demand Revalidation using the App Router and revalidatePath does not revalidate data on static paths.
When running the project locally, the revalidation correctly updates the data, but once deployed to Vercel, the data never gets updated. I have confirmed everything works fine when running locally with
next build && next start
Expected Behavior
I expect the data revalidation to successfully revalidate the home page of the app.
Which browser are you using? (if relevant)
No response
How are you deploying your application? (if relevant)
Vercel
The text was updated successfully, but these errors were encountered: