Skip to content

Conversation

Varixo
Copy link
Member

@Varixo Varixo commented Aug 13, 2025

Correctly find contexts with falsy values

@Varixo Varixo self-assigned this Aug 13, 2025
@Varixo Varixo requested a review from a team as a code owner August 13, 2025 14:23
Copy link

changeset-bot bot commented Aug 13, 2025

🦋 Changeset detected

Latest commit: 08c3a0c

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

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

@Varixo Varixo moved this from Backlog to Waiting For Review in Qwik Development Aug 13, 2025
@Varixo Varixo force-pushed the v2-context-falsy-value branch from 9541e78 to 08c3a0c Compare August 13, 2025 14:53
Copy link
Member

@wmertens wmertens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@wmertens wmertens enabled auto-merge August 13, 2025 15:03
@wmertens wmertens merged commit de199e8 into build/v2 Aug 13, 2025
17 checks passed
@wmertens wmertens deleted the v2-context-falsy-value branch August 13, 2025 15:04
@github-project-automation github-project-automation bot moved this from Waiting For Review to Done in Qwik Development Aug 13, 2025
Copy link

pkg-pr-new bot commented Aug 13, 2025

Open in StackBlitz

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

commit: 08c3a0c

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

[🐞] Actual value for useContext() can not be found, when using falsy context value
2 participants