Repository navigation
Conversation
|
Warning Review paused — included plan limit reachedKeep your review moving with free on-demand reviews.
On-demand reviews are free for the next 14 days. Promotion and pricing detailsOn-demand reviews are free for the next 14 days. After that, they cost $0.25 per reviewed file. Review limit detailsOr wait 14 minutes for your next included review. Limit details: You’ve used all 2 included reviews currently available. Review configuration: ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Advanced Run ID: 📒 Files selected for processing (4)
Comment |
|
Checked against ComfyUI_frontend |
|
Superseded. The Provider field fix for |
@lin-bot23
Small accuracy fix on the Comfy Cloud private model import page.
What changed
cloud/import-models.mdx(EN/zh/ja/ko), section Save your API keys:Why
The page previously said only "Click to add your API keys (you can name the key anything you like)". The Provider field is required: submitting without it fails with "Provider is required", and the provider is what enables automatic token usage when importing from that platform. One key per provider is enforced by the dialog (providers already configured are disabled in the select) and by the API (
DUPLICATE_PROVIDER).Verification
origin/main:src/platform/secrets/composables/useSecretForm.ts(validate()rejects an empty provider viasecrets.errors.providerRequired, disables providers already saved,secretValueHint= encrypted and cannot be viewed again, edit mode replaces the value) andsrc/platform/secrets/components/SecretFormDialog.vue.pnpm translate:sync-hash -- cloud/import-models.mdx: onlytranslationSourceHashplus the changed block hash were refreshed in zh/ja/ko.<Note>tags balanced.