-
-
Notifications
You must be signed in to change notification settings - Fork 104
api: Unencrypted group creation (#6927) #6932
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
Conversation
We also need API to list email contacts. Otherwise there is no way to create anything other than a group with only self. |
285d80a
to
416131b
Compare
60a7ba4
to
e4b14e2
Compare
Done in #6958 |
e4b14e2
to
349770b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the code lgtm as far as i can tell.
i tested the implementation already at deltachat/deltachat-ios#2770 - there are some screenshots - works like a charm.
thanks a lot!
for the api wording: from the angle that the core api serve the UI and an is very close to its needs and flows, |
personally I like |
`chat::create_group_ex()` gains an `encryption: Option<ProtectionStatus>` parameter to support unencrypted chats.
349770b
to
3286cb3
Compare
I think that even in UIs the new feature should be named "New {plaintext,unencrypted} email" because it's for users who set up Delta Chat with their existing mailboxes and know that Delta Chat sends emails, so it should be clear how it's different from a "usual" encrypted email. Let's wait for more opinions a bit and merge this. |
I also like the API name |
Let's discuss the naming here. At the Core level we don't need two separate APIs, so
create_group_ex()
or smth similar should work. The question is the naming in json-rpc (and CFFI if we decide to add it). Alternatives:create_email_group_chat()
or justcreate_email_chat()
(the word "chat" is needed to make it clear that a chat is created, not a message. Actually chat ID is returned). We don't use the word "email" for this purpose in the code currently, but maybe we want to switch to it. I'm not involved in the discussions a lot, so can't say much. But my "external" opinion is that technically everything is email and using this word only for a subset of chats/messages would complicate undestanding the code for newcomers and external audit. But if we switch to "email", probably we need to get rid of "adhoc" in the code.create_adhoc_{group,chat}()
. We already have "adhoc" inreceive_imf
at least.For now i named it
create_group_chat_unencrypted()
because i'm used to namespace/prefix naming personally and there are alreadyChat::is_encrypted()
in Core and{FullChat,BasicChat}::is_encrypted
in json-rpc which exactly reflect the difference betweencreate_group_chat()
and the new function and i decided not to introduce new words.closes #6927