Skip to content

Adapt settings#6313

Closed
nicodh wants to merge 3 commits into
mainfrom
adapt-settings
Closed

Adapt settings#6313
nicodh wants to merge 3 commits into
mainfrom
adapt-settings

Conversation

@nicodh

@nicodh nicodh commented Apr 29, 2026

Copy link
Copy Markdown
Member

Update settings

Add a configuration for imap folder. Not sure what would be an appropriate wording here

edit:
resolves #6333

  • imap folder UI is removed

  • removed "Move automatically to DeltaChat Folder"

  • removed "Only Fetch from DeltaChat Folder"

  • removed "Show Classic Emails"

  • removed "Delete Messages from Server"

  • add "force unencrypted"

@nicodh nicodh added the wait-for-core Waiting for an related core issue to be resolved label Apr 29, 2026
@r10s

This comment was marked as outdated.

@r10s r10s marked this pull request as draft April 30, 2026 14:29
@r10s

r10s commented Apr 30, 2026

Copy link
Copy Markdown
Member

converted to draft, see recent discussion in release coordination, changes probably needed:

  • imap folder should not be exposed in the UI
  • remove delete_server_after option
  • add "show unencrypted" option (name is not settled)

i suggest to postpone this PR until we have a core

nicodh added 2 commits May 14, 2026 18:36
- remove mvbox_move, only_fetch_mvbox, show_emails
- remove ImapFolderHandling dialog
- remove ShowClassicEmail dialog
@nicodh nicodh marked this pull request as ready for review May 15, 2026 20:30
@r10s

r10s commented May 16, 2026

Copy link
Copy Markdown
Member

ftr, and as discussed in one-to-one, we do not want to have the "force encrypted" option that much in the face. it does not work for most user, and we will otherwise get pushback that one can that easily lower security standard (tho it will not work :).

only saying, it is no blocker for this PR, and it is not yet fully decided where it will go (maybe to "advanced / relays / three dot menu" or "advanced / relay / edit" tho this is more per-relay)

@nicodh nicodh marked this pull request as draft May 20, 2026 06:43
@Simon-Laux

Copy link
Copy Markdown
Member

This is needed to run current core main branch with desktop, without this branch or removing code in working copy it does not run correctly.

@r10s

r10s commented May 21, 2026

Copy link
Copy Markdown
Member

ftr, and as discussed in one-to-one, we do not want to have the "force encrypted" option that much in the face. it does not work for most user, and we will otherwise get pushback that one can that easily lower security standard (tho it will not work :).

the option shall go to "Settings / Relays / Edit", see updated chatmail/core#8220 and deltachat/deltachat-android#4431

@nicodh

nicodh commented May 23, 2026

Copy link
Copy Markdown
Member Author

closed in favour of #6409 and #6410

@nicodh nicodh closed this May 23, 2026
@link2xt

link2xt commented May 23, 2026

Copy link
Copy Markdown
Collaborator

#6409 did not add "force encryption" setting, still need to add it somewhere.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

wait-for-core Waiting for an related core issue to be resolved

Projects

None yet

Development

Successfully merging this pull request may close these issues.

remove legacy options

4 participants