Fix/cache prod

[technophile-04]

Issue:

NOTE: This only occurs in prod, since there next does caching and build static stuff their.

Demo of admin page not refetching after mutation

https://github.com/BuidlGuidl/grants.buidlguidl.com/assets/80153681/2eb751a3-ffb1-4e2b-80fb-77abec506bd5

As we can once I make approve action, things are remaining as it is and not refetching the data.

Reason for above cause:

GET /api/grants/review route handler was static.

• In NextJS app router, route handlers with GET are by default static.
• Which means we only run await getAllGrantsForReview() in that api once in lifetime just during build time. After that /api/grants/review just returns the static data (All review grants present at build time).

NextJS marking api/grants/review as static during build time


Solution:

We need to opt out of caching for GET /api/grants/review route handler.Since we want admin to see updated data as soon as he does mutation.

The way we OPT out of caching for GET is checkout this in docs.

Hence in a4d0c400cc97b369278f87fd19b1e2cf7284749b we make use of request (which makes GET handler as dynamic) and it also reavalidatePath /admin(which was also static). The example was taken from here in docs

[vercel[bot]]

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

Name	Status	Preview	Comments	Updated (UTC)
grants-bg	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Mar 6, 2024 9:41am

[carletex]

Thanks for this Shiv! This is forcing me to learn a bunch about NextJS caching, which is not simple :D

I left a couple of comments.

---

Sad that this only works for "fetch". With fetch, it shows HIT / MISS on the dev console so there is no need to build & start.

https://nextjs.org/docs/app/api-reference/next-config-js/logging

[technophile-04]

Tysm, updated 🙌

This is forcing me to learn a bunch about NextJS caching, which is not simple :D

Yeah lol it's not all straightforward :( and docs seem to be kind of distributed too, also the thing it works differently in the dev environment as compared to prod is bad :(

Sad that this only works for "fetch". With fetch, it shows HIT / MISS on the dev console so there is no need to build & start.

Yeah :(