-
Notifications
You must be signed in to change notification settings - Fork 140
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.
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
feat: Add more server side arguments #1846
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please link the electron side change for these params, as currently these params are also set by the electron side explicitly. Currently electron side values will take priority.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you link the correct change in the desktop repo? Currently the link points to this specific frontend repo PR.
resolves Comfy-Org/desktop#424
desktop dependency: #1846
0b96028
to
cc60713
Compare
Changes made:
On the electron side support configuring of host and port: |
resolves Comfy-Org/desktop#424
desktop dependency: Comfy-Org/desktop#465