Drive Sync now pending UX + error alerts (#90) #91

Merged
westfarn merged 1 commits from feature/drive-sync-error-ux-90 into master 2026-08-02 04:07:22 -07:00
Owner

Summary

  • Closes #90
  • Depends on backend chat_backend#57 (202 + pending status)
  • Sync now: enqueue, poll until status leaves pending, show error/success alert
  • Per-connection last_sync_error shown when status is error
  • OAuth green “Drive connected” banner unchanged (connect-only)
  • Refreshes document list on successful sync

Test plan

  • drive.test + DocumentStoragePage.test
  • Manual with backend #57: Sync fails (Drive API off) → red alert with truncated error
  • Manual: Sync succeeds → success alert + docs refresh
  • Manual: Syncing… / button disabled while pending
## Summary - Closes [#90](https://git.aimloperations.com/ai_ml_operations/chat_web_app/issues/90) - Depends on backend [chat_backend#57](https://git.aimloperations.com/ai_ml_operations/chat_backend/issues/57) (202 + pending status) - Sync now: enqueue, poll until status leaves `pending`, show **error/success alert** - Per-connection `last_sync_error` shown when status is `error` - OAuth green “Drive connected” banner unchanged (connect-only) - Refreshes document list on successful sync ## Test plan - [x] `drive.test` + `DocumentStoragePage.test` - [ ] Manual with backend #57: Sync fails (Drive API off) → red alert with truncated error - [ ] Manual: Sync succeeds → success alert + docs refresh - [ ] Manual: Syncing… / button disabled while pending
westfarn added 1 commit 2026-08-02 04:02:50 -07:00
Surface Drive sync pending/error state after Sync now (#90).
Unit Tests / test (pull_request) Successful in 10s
9224d0cf45
Poll until last_sync_status leaves pending, show error/success alerts,
and keep the OAuth connected banner separate from sync outcome.
westfarn merged commit 6c96c50d99 into master 2026-08-02 04:07:22 -07:00
westfarn deleted branch feature/drive-sync-error-ux-90 2026-08-02 04:07:22 -07:00
Sign in to join this conversation.
No Reviewers
No labels
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: ai_ml_operations/chat_web_app#91