-
[OPIK-3958] [FE] Simplify Ollama provider connection flow (#5005)
frostbyte_neo 发布于
2026-02-02 12:04:52 +00:00 - [OPIK-3958] [FE] Simplify Ollama provider connection flow
- Remove redundant 'Discover Models' button
- Rename 'Test Connection' to 'Connect'
- Auto-fetch models after successful connection
- Merge loading states into single button
- Revision 2: Address PR review comments from JetoPistola
- Disable button during model discovery to prevent duplicate requests
- Change button text to 'Connection failed (click to retry)' for clearer UX
- Track model fetch failures separately from connection status
- Show 'Connected (models failed)' when connection succeeds but models fail to load
- Add orange checkmark icon for partial success state
Addresses all review comments on PR #5005:
- Comment 1 (blocking): Disabled condition now includes listModelsMutation.isPending
- Comment 2: Added retry hint to failed connection state
- Comment 3: Button now reflects model discovery failures with dedicated state
- Revision 3: Use theme-aware color token for dark mode support
- Replace text-orange-600 with text-warning design system token
- Ensures icon remains visible in dark mode
- Addresses baz-reviewer comment on PR #5005
Co-authored-by: baz-reviewer[bot]
下载附件