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

Archived in project

Development

Successfully merging this pull request may close these issues.

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

4 participants