Skip to content

feat: Backup tokens with the Block Store API - #3073

Merged
LouisCAD merged 3 commits into
mainfrom
block-store
Sep 24, 2026
Merged

LouisCAD merged 3 commits into
mainfrom
block-store

Conversation

@LouisCAD

@LouisCAD LouisCAD commented Sep 17, 2026 •

Copy link
Copy Markdown
Contributor

Copilot AI balanced review requested due to automatic review settings September 17, 2026 07:53
@github-actions github-actions Bot added the dependent This MR depends on another PR label Sep 17, 2026
@LouisCAD
LouisCAD requested a review from tevincent September 17, 2026 07:53

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

The dependent Core implementation reports success before asynchronous Block Store writes complete.

Get a fresh assessment by requesting another Copilot review.

Pull request overview

Adds Block Store token backup to the app’s full-backup flow.

Changes:

  • Wraps full backups with Core’s Block Store token backup helper.
  • Preserves existing device-transfer Realm backup behavior.
File summaries
File Description
MailFullBackupAgent.kt Integrates Block Store token backup.
Review details
  • Files reviewed: 2/2 changed files
  • Comments generated: 1
  • Review effort level: Balanced

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread app/src/main/java/com/infomaniak/mail/backup/MailFullBackupAgent.kt Outdated
@github-actions github-actions Bot removed the dependent This MR depends on another PR label Sep 24, 2026
@github-actions

Copy link
Copy Markdown

This PR/issue depends on:

@LouisCAD
LouisCAD enabled auto-merge September 24, 2026 12:53
@sonarqubecloud

Copy link
Copy Markdown

@LouisCAD
LouisCAD merged commit 38bb3c5 into main Sep 24, 2026
10 checks passed
@LouisCAD
LouisCAD deleted the block-store branch September 24, 2026 12:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants