Skip to content

fix(router): reuse SSR-rendered loader data when the first fetch eTag matches - #9221

Merged
Varixo merged 1 commit into
mainfrom
fix/loader-first-fetch-etag
Nov 3, 2026
Merged

Varixo merged 1 commit into
mainfrom
fix/loader-first-fetch-etag

Conversation

@Varixo

@Varixo Varixo commented Oct 17, 2026

Copy link
Copy Markdown
Member

Loaders with serializationStrategy 'never' fetch their data on the first client read, which reran every subscriber even when the data was unchanged. When the loader has an eTag option, SSR now stores the same eTag the q-loader endpoint sends, and a matching first fetch adopts the value without notifying.

… matches

Loaders with serializationStrategy 'never' fetch their data on the first client
read, which reran every subscriber even when the data was unchanged. When the
loader has an eTag option, SSR now stores the same eTag the q-loader endpoint
sends, and a matching first fetch adopts the value without notifying.
@Varixo Varixo self-assigned this Oct 17, 2026
@Varixo
Varixo requested a review from a team as a code owner October 17, 2026 11:18
@changeset-bot

changeset-bot Bot commented Oct 17, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: d73c60a

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 8 packages
Name Type
@qwik.dev/router Patch
eslint-plugin-qwik Patch
@qwik.dev/mcp Patch
create-qwik Patch
@qwik.dev/devtools Patch
@qwik.dev/react Patch
@qwik.dev/utils Patch
@qwik.dev/core Patch

Not sure what this means? Click here to learn what changesets are.

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

@maiieul maiieul moved this to Waiting For Review in Qwik Development Oct 17, 2026
@pkg-pr-new

pkg-pr-new Bot commented Oct 17, 2026

Copy link
Copy Markdown

Open in StackBlitz

@qwik.dev/core

npm i https://pkg.pr.new/QwikDev/qwik/@qwik.dev/core@9221

@qwik.dev/router

npm i https://pkg.pr.new/QwikDev/qwik/@qwik.dev/router@9221

@qwik.dev/utils

npm i https://pkg.pr.new/QwikDev/qwik/@qwik.dev/utils@9221

eslint-plugin-qwik

npm i https://pkg.pr.new/QwikDev/qwik/eslint-plugin-qwik@9221

create-qwik

npm i https://pkg.pr.new/QwikDev/qwik/create-qwik@9221

@qwik.dev/optimizer

npm i https://pkg.pr.new/QwikDev/qwik/@qwik.dev/optimizer@9221

@qwik.dev/devtools

npm i https://pkg.pr.new/QwikDev/qwik/@qwik.dev/devtools@9221

@qwik.dev/mcp

npm i https://pkg.pr.new/QwikDev/qwik/@qwik.dev/mcp@9221

commit: d73c60a

@github-actions

github-actions Bot commented Oct 17, 2026 •

Copy link
Copy Markdown
Contributor
built with Refined Cloudflare Pages Action

⚡ Cloudflare Pages Deployment

Name Status Preview Last Commit
qwik-docs ✅ Ready (View Log) Visit Preview d73c60a

@Varixo
Varixo merged commit becd5e2 into main Nov 3, 2026
55 checks passed
@Varixo
Varixo deleted the fix/loader-first-fetch-etag branch November 3, 2026 20:26
@github-project-automation github-project-automation Bot moved this from Waiting For Review to Done in Qwik Development Nov 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant