Comfy-Org / Comfy-Org/ComfyUI

False info in README.md

Open Beginner friendly
#15,223 3 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
133k
Forks
15.7k
Avg merge
1d 7h
Merged PRs (30d)
158

Description

This table says "implies `--enable-manager`", but I spent a good hour reinstalling, pulling, restoring backups and more trying to get `--enable-manager-legacy-ui` to work.

But in the end the solution was that I needed to add **both** flags for legacy ui to show.

`--enable-manager-legacy-ui` alone doesn't even make manager settings visible in options.

| Flag | Description |
|------|-------------|
| `--enable-manager` | Enable ComfyUI-Manager |
| `--enable-manager-legacy-ui` | Use the legacy manager UI instead of the new UI (implies `--enable-manager`) |
| `--disable-manager-ui` | Disable the manager UI and endpoints while keeping background features like security checks and scheduled installation completion (requires `--enable-manager`) |

Contributor guide

Open the contributing guide

Research direction

Update the flag table in README.md, focusing on the descriptions for --enable-manager and --enable-manager-legacy-ui. Verify that the documentation states both flags are needed for the legacy UI and that the legacy flag alone does not enable manager settings visibility.

Written by the indexing model from the issue text.

Assessment

Domain
documentation
Issue type
Documentation
Difficulty
1/5
Estimated time
Under an hour
Activity status
Quiet
Clarity
Clearly specified
Newbie friendliness
85/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.