Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove all IMAP-Folder-Handling options for chatmail accounts #5381

Closed
hpk42 opened this issue Mar 26, 2024 · 4 comments
Closed

Remove all IMAP-Folder-Handling options for chatmail accounts #5381

hpk42 opened this issue Mar 26, 2024 · 4 comments

Comments

@hpk42
Copy link
Contributor

hpk42 commented Mar 26, 2024

Chatmail addresses do not need any special imap-folder settings and it makes sense, also for the server-side chatmail setup, to not have any folders because we don't need it, and fresh instant-onboarding users don't need to think about it even in advanced settings. Not sure how to best signal to UIs to not expose these options, maybe similar to #5380 with a "NOFOLDERS" capability.

@Simon-Laux
Copy link
Member

maybe just a readonly config key that the ui can read

@iequidoo
Copy link
Collaborator

iequidoo commented Apr 3, 2024

Maybe add a function Context::is_configurable() that should be checked for options that are not always configurable?

@r10s
Copy link
Member

r10s commented May 3, 2024

Maybe add a function Context::is_configurable() that should be checked for options that are not always configurable?

i think, this is a bit over the top, eg. having 4 states that could in theory be all on/off in all combinations. UI would need to think over how to hide a group and whatnot then.

i would go for a single switch, and yes, reusing get_config() for that seems to be a pragmatic, simple approach not requiring new API - also, we have already read-only options there, so that would not even be sth. new

@r10s
Copy link
Member

r10s commented May 17, 2024

@r10s r10s closed this as completed May 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

No branches or pull requests

4 participants