New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Cleanup] Remove unused GET internal/files API endpoint #6714

Merged

comfyanonymous merged 1 commit into master from remove_internal_files_api

Feb 5, 2025

Collaborator

huchenlei commented Feb 5, 2025

The GET internal/files API endpoint is currently unused by the frontend. This PR removes it.

Ref: #4295


          Remove unused GET /files API endpoint

1d56bc2

huchenlei requested review from yoland68, robinjhuang, webfiltered, pythongosssss, ltdrdata and Kosinkadink as code owners

February 5, 2025 18:51

Collaborator Author

huchenlei commented Feb 5, 2025

@robinjhuang PTAL

huchenlei changed the title ~~Remove unused GET internal/files API endpoint~~ [Cleanup] Remove unused GET internal/files API endpoint

robinjhuang approved these changes

View reviewed changes

comfyanonymous merged commit f1059b0 into master

6 checks passed

comfyanonymous deleted the remove_internal_files_api branch

February 5, 2025 23:48

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

robinjhuang robinjhuang approved these changes

yoland68 Awaiting requested review from yoland68 yoland68 is a code owner

webfiltered Awaiting requested review from webfiltered webfiltered is a code owner

pythongosssss Awaiting requested review from pythongosssss pythongosssss is a code owner

ltdrdata Awaiting requested review from ltdrdata ltdrdata is a code owner

Kosinkadink Awaiting requested review from Kosinkadink Kosinkadink is a code owner

Labels

None yet