Skip to content
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

fix: don't execute QRLs for elements marked as deleted #7448

Draft
wants to merge 1 commit into
base: build/v2
Choose a base branch
from

Conversation

Varixo
Copy link
Member

@Varixo Varixo commented Mar 22, 2025

No description provided.

Copy link

changeset-bot bot commented Mar 22, 2025

🦋 Changeset detected

Latest commit: 6d1345f

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

This PR includes changesets to release 5 packages
Name Type
@qwik.dev/core Patch
eslint-plugin-qwik Patch
@qwik.dev/react Patch
@qwik.dev/router Patch
create-qwik 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

Copy link

pkg-pr-new bot commented Mar 22, 2025

Open in Stackblitz

npm i https://pkg.pr.new/QwikDev/qwik/@qwik.dev/core@7448
npm i https://pkg.pr.new/QwikDev/qwik/@qwik.dev/router@7448
npm i https://pkg.pr.new/QwikDev/qwik/eslint-plugin-qwik@7448
npm i https://pkg.pr.new/QwikDev/qwik/create-qwik@7448

commit: 6d1345f

Copy link
Contributor

github-actions bot commented Mar 22, 2025

built with Refined Cloudflare Pages Action

⚡ Cloudflare Pages Deployment

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

@Varixo Varixo force-pushed the v2-vnode-refs branch 2 times, most recently from 1603e0a to 9952e76 Compare March 22, 2025 20:12
@Varixo Varixo changed the title chore: add vNodeRefs chore: add clientVNodeRefs Mar 22, 2025
@Varixo Varixo changed the title chore: add clientVNodeRefs fix: don't execute QRLs for elements marked as deleted Mar 23, 2025
@Varixo Varixo force-pushed the v2-vnode-refs branch 2 times, most recently from 3a8657a to cd43144 Compare March 25, 2025 20:38
@Varixo Varixo marked this pull request as ready for review March 25, 2025 20:39
@Varixo Varixo requested a review from a team as a code owner March 25, 2025 20:39
@Varixo Varixo marked this pull request as draft March 29, 2025 19:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Waiting For Review
Development

Successfully merging this pull request may close these issues.

1 participant