Skip to content

feat: add device message about legacy mvbox_move#7777

Merged
r10s merged 2 commits intomainfrom
link2xt/mvbox_move-device-message
Feb 4, 2026
Merged

feat: add device message about legacy mvbox_move#7777
r10s merged 2 commits intomainfrom
link2xt/mvbox_move-device-message

Conversation

@link2xt
Copy link
Collaborator

@link2xt link2xt commented Jan 26, 2026

Closes #7774

@link2xt link2xt force-pushed the link2xt/mvbox_move-device-message branch from b2886e8 to 9e5d86d Compare January 26, 2026 23:09
@r10s
Copy link
Contributor

r10s commented Jan 28, 2026

i drafted a page with more details at deltachat/deltachat-pages#1254

@Hocuri
Copy link
Collaborator

Hocuri commented Feb 4, 2026

The test fails because the default for MvboxMove is still 1. We should change the default to 0.

Edit: Actually, even though the default is "1" here for compatibility reasons, is disabled in new profiles during configure already. So, we only need to fix the failing Rust test, which should be pretty easy.

Copy link
Collaborator

@Hocuri Hocuri left a comment

Choose a reason for hiding this comment

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

Device message LGTM once Björn's comment is addressed. (and we need to make CI pass, ofc)

src/sql.rs Outdated

/// Adds device message about `mvbox_move` config deprecation
/// if the user has it enabled.
async fn add_mvbox_move_deprecation_message(context: &Context) -> Result<()> {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Should be called maybe_add_mvbox_move_deprecation_message(), because this is how we usually call functions with a condition inside

@r10s
Copy link
Contributor

r10s commented Feb 4, 2026

i went forward and changed then device message, after discussing with @Hocuri

@r10s
Copy link
Contributor

r10s commented Feb 4, 2026

Actually, even though the default is "1" here for compatibility reasons, is disabled in new profiles during configure already. So, we only need to fix the failing Rust test, which should be pretty easy. So, we only need to fix the failing Rust test, which should be pretty easy.

the failing test is test_reaction_summary. for some reason, it sees the device message where it should not, if MvboxMove is not set for new profiles. maybe a race, running housekeeping before configure() sets the value? EDIT: housekeeping is run by the test itself - not sure, why mvbox_move is 1 there, when it is set to 0 during configure()

EDIT: if you change the last lines of the test to ...

        assert!(alice.is_configured().await?);
        assert_eq!(alice.get_config(Config::MvboxMove).await?.unwrap(), "0"); // XXX this fails
        assert_summary(&alice, "kewl").await;

        Ok(())

... the test fails at XXX. so, MvboxMove is actually not set to "0" for that test.

@link2xt link2xt force-pushed the link2xt/mvbox_move-device-message branch from b81bae3 to 0e7efa1 Compare February 4, 2026 18:12
@link2xt link2xt force-pushed the link2xt/mvbox_move-device-message branch from 0e7efa1 to fb4c56f Compare February 4, 2026 18:13
src/stock_str.rs Outdated
ChatUnencryptedExplanation = 230,

#[strum(props(
fallback = "You are using the legacy option \"Settings → Advanced → Move automatically to DeltaChat Folder\".\n\nThis option will be removed in a few weeks and you should disable it already today.\n\nIf having chat messages mixed into your inbox is a problem, see https://delta.chat/legacy-move how to move to a dedicated chat profile"
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

"move to a dedicated chat profile" sounds wrong, the profile stays the same but you move to a different relay.

Copy link
Contributor

Choose a reason for hiding this comment

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

the outcome by adding a relay and removing the existing one, however, is a dedicated chat profile. which is needed as a shared chat profile is no longer supported

here, however, maybe we can just strike the suffix, so If having chat messages mixed into your inbox is a problem, see https://delta.chat/legacy-move

@link2xt link2xt force-pushed the link2xt/mvbox_move-device-message branch from fb4c56f to 1113e0d Compare February 4, 2026 18:56
@link2xt link2xt marked this pull request as ready for review February 4, 2026 18:56
@link2xt link2xt force-pushed the link2xt/mvbox_move-device-message branch from 1113e0d to 1459ba2 Compare February 4, 2026 19:01
@r10s r10s merged commit 00e78ee into main Feb 4, 2026
30 checks passed
@r10s r10s deleted the link2xt/mvbox_move-device-message branch February 4, 2026 20:51
@r10s
Copy link
Contributor

r10s commented Feb 4, 2026

ftr, this is how it looks like:

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.

Add device message for users having mvbox_move enabled

3 participants