Repository navigation
Fix account settings and add seamless deletion and recovery - #73
Conversation
There was a problem hiding this comment.
Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.
An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing.
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: c001408166
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Codecov Report❌ Patch coverage is
📢 Thoughts on this report? Let us know! |
Fix account settings so Google reflects the server's linked identities, disable Change password, Two-factor authentication, and Active sessions, and add account deletion with a 30-day recovery window.
Deletion completes from the existing signed-in session using the standard confirmation dialog. Signing in to an account awaiting deletion offers Keep account, which restores access and opens the library without a second login or a success snackbar. The Google recovery page uses the welcome illustration behind the dialog. Token and profile-switch guards prevent late deletion, refresh, or recovery responses from replacing the active account.
Requires server #40. Deploy the server first because this client requires
connected_providersin user responses and the new deletion/recovery endpoints. Policy and API documentation: docs #85.Validation after rebasing onto current development: full Flutter suite passed (1,762 tests, 19 skipped), formatter/analyzer/bootstrap checks passed, and the web build passed. Focused coverage includes deletion/recovery, retries, expired proofs, and server-profile switches. Live provider smoke tests and production deployment were not run.
Related: FR-1_6, UC-1_6. The app shows the 30-day window before confirmation and the deletion date afterward, with the exact dated preview still tracked as a requirement gap.