Conversation
modified: install_by_id - Change the install path of the CNR node added by the system to be based on the repo URL instead of the CNR ID.
POST /v2/manager/queue/batch
GET /v2/manager/queue/history_list
GET /v2/manager/queue/history?id={id}
GET /v2/manager/queue/abort_current
modified: install_by_id - Change the install path of the CNR node added by the system to be based on the repo URL instead of the CNR ID.
This was referenced Apr 12, 2025
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
…ructure. The glob/... version will be applied later after the cacheless implementation is completed.
* [feat] Add bulk import failure info API endpoint - Add import_fail_info_bulk endpoint to both glob and legacy manager servers - Supports bulk processing of cnr_ids and urls arrays in single request - Maintains same error handling pattern as original import_fail_info API - Reduces API calls from N to 1 for conflict detection optimization - Validates input parameters and provides proper error responses 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com> * modified: remove manager button completely. Now, even when using the legacy UI, it must always be accessed through the menu. * chore(api): Add temporary cache reload for import_fail_info_bulk --------- Co-authored-by: Claude <noreply@anthropic.com> Co-authored-by: Dr.Lt.Data <dr.lt.data@gmail.com>
- Regenerated Pydantic models from updated OpenAPI specification - Updated import_fail_info_bulk route handler to use ImportFailInfoBulkRequest/Response models - Replaced manual JSON validation with Pydantic model validation - Added proper error handling with ValidationError - Updated data_models/__init__.py to export new models Following the process outlined in data_models/README.md for type safety and consistency.
…k-api Add Pydantic validation to import_fail_info_bulk endpoint
modified: The matrix share feature is now only available when the `matrix-nio` dependency is installed. If `matrix-nio` is not installed: 1. Apply a strikethrough to the matrix checkbox text in the share UI and display a tooltip. 2. A warning is logged at startup indicating that `matrix-nio` is missing, along with the installation command. fixed: Corrected an issue where PR #2025 was merged into draft-v4 but applied only to `legacy/..` and not to `glob/..`
…ckend structure." This reverts commit 121a5a1.
Revert "As a temporary measure, the new UI will use the legacy/... ba…
* fix: import failed info bulk api bug fix * fix: Remove unused ImportFailInfoBulkResponse import
…he default channel - Mismatch issue between ltdrdata/ and Comfy-Org/ modified: /v2/customnode/installed – cnr_id was being returned in a normalized form modified: /v2/customnode/installed – when both an enabled nodepack and a disabled nodepack existed, modified to report only the enabled nodepack fixed: Removed unnecessary warning messages printed during nodepack installation
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
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.
This PR is meant to be paired with:
This PR is based on branch:
TODO:
--enable-manager-legacy-ui--disable-managerargcomfyui-frontend-packagewas notTest: UsingThis is not worth supporting--if the user wants the legacy version they can use--disable-managerarg then loading manager as a custom node--enable-manager-legacy-ui