Manager/compatibility #4492
Draft
Manager/compatibility #4492
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
|
comfy-jinyi
force-pushed
the
manager/menu-items-migration
branch
from
93473bc to
df67cdd
Compare
Updated ComfyUI Manager types and API schemas to support the new menu items structure and manager functionality.
Updated ComfyUI Manager service and dialog service to support the new menu items structure and API endpoints.
Updated manager queue composable, server logs composable, workflow packs composable, and manager store to support the new manager API structure and state management patterns.
Updated manager dialog components, pack cards, version selectors, and action buttons to work with the new manager API and state management structure.
Updated tests for manager queue composable, server logs composable, and manager store to work with the new API signatures and functionality.
- Fix ManagerProgressDialogContent test mock to include all required store methods - Add missing MergedNodePack, RegistryPack type definitions and isMergedNodePack type guard - Ensure all unit tests (548 passed) and component tests (174 passed) are working - Fix TypeScript compilation errors related to manager store interfaces
Update test to simulate cm-task-started events before logs events to match the actual behavior of the composable.
The test was expecting subscribeLogs(false) to be called, but this was commented out in commit 33d6447 to fix logs stopping after the first of multiple queue tasks. Updated test to reflect this temporary change.
- Add PackUpdateButton component for bulk updates - Add useUpdateAvailableNodes composable to track available updates - Integrate update all button in RegistrySearchBar - Add localization strings for update functionality - Add comprehensive tests for update functionality - Add loading state to PackActionButton
Co-authored-by: Claude <[email protected]>
comfy-jinyi
force-pushed
the
manager/compatibility
branch
from
72c2038 to
0a2f2d8
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
┆Issue is synchronized with this Notion page by Unito