Conversation
|
|
✅ Deploy Preview for nuxt3-docs canceled.
|
|
I was too quick - looks like the test fixture is rightly finding an issue. |
|
It is a breaking change coming with this PR indeed. When navigating on client-side, the initial payload cache won't be used as option is dropped. Later will add actual cache support as mentioned in #3985 |
|
As I understand it, that will basically break the payloadExtraction implementation because this will only be used on initial load (which was the case even if payloads were not extracted). If we merge this without fixing this, I think we should also disable |
672fb1e to
33eb38b
Compare
|
Note: now that I think about it, I think we should disable payload extraction by default anyway as I think there's some more refinement/discussion to do, particularly around |
|
Updated implementation to keep CSR payloads permanently (like initialCache: true) in |
Relates to nuxt/framework#8885 Relates to nuxt/framework#8917
🔗 Linked issue
❓ Type of change
📚 Description
Race condition issue in #3985 seems to be resolved. Removing initialCache option and only use utility to preserve payload data while hydrating.
BREAKING CHANGE: Payload cache won't be used when doing client-side navigation back to same page and fresh data will be fetched.
📝 Checklist