Skip to content

feat: add backup versions to the importing error message#7864

Merged
link2xt merged 1 commit intomainfrom
link2xt/rytkvqslzmlv
Feb 19, 2026
Merged

feat: add backup versions to the importing error message#7864
link2xt merged 1 commit intomainfrom
link2xt/rytkvqslzmlv

Conversation

@link2xt
Copy link
Copy Markdown
Collaborator

@link2xt link2xt commented Feb 16, 2026

This would have helped debugging the problem reported at https://support.delta.chat/t/backup-too-new-please-update-delta-chat-message/4761

src/imex.rs Outdated
ensure!(
version <= DCBACKUP_VERSION,
"Backup too new, please update Delta Chat"
"Backup version {version} is too new, the latest supported is {DCBACKUP_VERSION}, please update the application"
Copy link
Copy Markdown
Collaborator

@Hocuri Hocuri Feb 17, 2026

Choose a reason for hiding this comment

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

I don't think we should replace "Delta Chat" with "the application" here. Grepping for "delta" in all *.rs files yields 734 results, which will need a dedicated effort to replace (and a stock string for the application name). Replacing it in some places will just make this harder, and calling it "the application" is probably confusing.

In order to clearly tell the user what's the problem and what they should do:

Suggested change
"Backup version {version} is too new, the latest supported is {DCBACKUP_VERSION}, please update the application"
"This profile is from a newer version of Delta Chat.
Please update Delta Chat and try again.
(Profile version: v{version}; supported: v{DCBACKUP_VERSION})"

@link2xt link2xt force-pushed the link2xt/rytkvqslzmlv branch from d528168 to bf59e2a Compare February 17, 2026 22:04
@link2xt link2xt merged commit e95dca8 into main Feb 19, 2026
30 checks passed
@link2xt link2xt deleted the link2xt/rytkvqslzmlv branch February 19, 2026 15:28
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.

2 participants