Allow POST /userdata/{file} endpoint to return full file info#5446
Merged
comfyanonymous merged 5 commits intomasterfrom Nov 4, 2024
Merged
Allow POST /userdata/{file} endpoint to return full file info#5446comfyanonymous merged 5 commits intomasterfrom
/userdata/{file} endpoint to return full file info#5446comfyanonymous merged 5 commits intomasterfrom
Conversation
tiangles
pushed a commit
to diffus-me/ComfyUI
that referenced
this pull request
Nov 29, 2024
…y-Org#5446) * Refactor listuserdata * Full info param * Add tests * Fix mock * Add full_info support for move user file
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
2 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 makes POST
/userdata/{file}and POST/userdata/{file}/move/{dest}accept an extra optional paramfull_infoso that the updated full file info (size and last modified timestamp) is returned to the caller. Without the param, the return value would be the file path string.This can save the frontend an extra GET
/userdatacall to update the size and last modified info.