Observed in #2808 DX review. Low priority refactor.
`dashboard-agents.html`'s connect form ships `platform_type` three separate times — once per auth-type field group (bearer/basic, OAuth auth-code, OAuth client-credentials). Each has a distinct `name=` suffix (`platform_type`, `platform_type_oauth`, `platform_type_cc`) so the active one's value is submitted.
The field is about the agent, not the auth method. Hoist it above the auth-type select so the operator fills it once, regardless of auth choice.
Proposal
- Render one `platform_type` select above the auth-type select.
- Remove the three duplicated selects from inside each field group.
- Update the three save handlers to read the single hoisted field.
Scope notes
- Small (<30 lines of HTML/JS).
- No behavior change from the server's perspective.
- Cleanest way is to do it alongside a follow-up UI refactor.
Related: #2808.
Observed in #2808 DX review. Low priority refactor.
`dashboard-agents.html`'s connect form ships `platform_type` three separate times — once per auth-type field group (bearer/basic, OAuth auth-code, OAuth client-credentials). Each has a distinct `name=` suffix (`platform_type`, `platform_type_oauth`, `platform_type_cc`) so the active one's value is submitted.
The field is about the agent, not the auth method. Hoist it above the auth-type select so the operator fills it once, regardless of auth choice.
Proposal
Scope notes
Related: #2808.