Skip to content

[backport rh-test] load assets browser before fetch completes and show loading state - #6236

Merged
christian-byrne merged 1 commit into
rh-testfrom
backport-6189-to-rh-test
Oct 23, 2025
Merged

christian-byrne merged 1 commit into
rh-testfrom
backport-6189-to-rh-test

Conversation

@comfy-pr-bot

@comfy-pr-bot comfy-pr-bot commented Oct 23, 2025 •

Copy link
Copy Markdown
Member

Backport of #6189 to rh-test

Automatically created by backport workflow.

┆Issue is synchronized with this Notion page by Unito

)

## Summary

Moves the fetch and post-fetch logic associated with the asset browser
into the component and shows a loading state while fetching.

To test, use this branch:
Comfy-Org/ComfyUI#10045



https://github.com/user-attachments/assets/718974d5-efc7-46a0-bcd6-e82596d4c389

┆Issue is synchronized with this [Notion
page](https://www.notion.so/PR-6189-load-assets-browser-before-fetch-completes-and-show-loading-state-2946d73d365081879d1bd05d86e8c036)
by [Unito](https://www.unito.io)

---------

Co-authored-by: GitHub Action <action@github.com>
@comfy-pr-bot comfy-pr-bot added the backport Backporting a PR onto a release candidate label Oct 23, 2025
@dosubot dosubot Bot added the size:XL This PR changes 500-999 lines, ignoring generated files. label Oct 23, 2025
@github-actions

github-actions Bot commented Oct 23, 2025 •

Copy link
Copy Markdown
Contributor

🎭 Playwright Test Results

❌ Some tests failed

⏰ Completed at: 10/23/2025, 09:32:54 PM UTC

📊 Test Reports by Browser


🎉 Click on the links above to view detailed test results for each browser configuration.

@christian-byrne
christian-byrne merged commit 5e13137 into rh-test Oct 23, 2025
11 of 24 checks passed
@christian-byrne
christian-byrne deleted the backport-6189-to-rh-test branch October 23, 2025 20:53
zeekay pushed a commit to hanzoui/frontend that referenced this pull request Jul 26, 2026
…w loading state (Comfy-Org#6236)

Backport of Comfy-Org#6189 to `rh-test`

Automatically created by backport workflow.

┆Issue is synchronized with this [Notion
page](https://www.notion.so/PR-6236-backport-rh-test-load-assets-browser-before-fetch-completes-and-show-loading-state-2956d73d3650817386fad4f54b1c0a89)
by [Unito](https://www.unito.io)

Co-authored-by: Christian Byrne <cbyrne@comfy.org>
Co-authored-by: GitHub Action <action@github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport Backporting a PR onto a release candidate size:XL This PR changes 500-999 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants