Skip to content

Conversation

@Varixo
Copy link
Member

@Varixo Varixo commented Aug 28, 2025

Async resolve function has been added. It should be used to avoid reruning code before asynccomputed.value during resolving a promise

@changeset-bot
Copy link

changeset-bot bot commented Aug 28, 2025

🦋 Changeset detected

Latest commit: 5432199

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 In progress in Qwik Development Aug 28, 2025
@Varixo Varixo moved this from In progress to Waiting For Review in Qwik Development Aug 28, 2025
@Varixo Varixo linked an issue Aug 28, 2025 that may be closed by this pull request
@Varixo
Copy link
Member Author

Varixo commented Aug 28, 2025

This PR is to discuss as it adds new public API for async computed

@Varixo Varixo force-pushed the v2-async-read-value branch from d84bb2a to 73e92e9 Compare August 28, 2025 19:07
@pkg-pr-new
Copy link

pkg-pr-new bot commented Aug 28, 2025

Open in StackBlitz

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

commit: 5432199

@github-actions
Copy link
Contributor

github-actions bot commented Aug 28, 2025

built with Refined Cloudflare Pages Action

⚡ Cloudflare Pages Deployment

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

error: Error | null;
loading: boolean;
// (undocumented)
resolve(): Promise<void>;
Copy link
Member

Choose a reason for hiding this comment

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

Let's make it return the value

Copy link
Member

Choose a reason for hiding this comment

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

Is this still valid?

@wmertens
Copy link
Member

@Varixo should we also add .resolved like in QRLs to get the latest value? An alias for untrackedValue basically.

@Varixo Varixo force-pushed the v2-async-read-value branch from 73e92e9 to 5432199 Compare October 27, 2025 18:53
@Varixo Varixo marked this pull request as ready for review October 27, 2025 18:57
@Varixo Varixo requested review from a team as code owners October 27, 2025 18:57
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 merged commit b14fe22 into build/v2 Oct 27, 2025
21 checks passed
@github-project-automation github-project-automation bot moved this from Waiting For Review to Done in Qwik Development Oct 27, 2025
@wmertens wmertens deleted the v2-async-read-value branch October 27, 2025 20:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[🐞] v2 solve problem with async computed used in tasks

3 participants