-
-
Notifications
You must be signed in to change notification settings - Fork 182
feat: Adapt to channel-invite QR codes #3886
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
base: main
Are you sure you want to change the base?
Conversation
Cloning wouldn't work, because you can only be added to a broadcast channel via a QR code
|
To test the changes in this pull request, install this apk: |
72d991d to
3378f77
Compare
|
To test the changes in this pull request, install this apk: |
|
To test the changes in this pull request, install this apk: |
| if (dcContext.sendMsg(dcChat.getId(), msg) == 0) { | ||
| Util.runOnMain(()-> Toast.makeText(ConversationActivity.this, dcContext.getLastError(), Toast.LENGTH_LONG).show()); | ||
| String lastError = dcContext.getLastError(); | ||
| if (!"".equals(lastError)) { |
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.
This check is so that we don't show an empty toast when sending a message into a legacy channel fails.
|
To test the changes in this pull request, install this apk: |
Companion PR for chatmail/core#7268
DC_QR_ASK_JOIN_BROADCAST = 204(CFFI) /AskJoinBroadcast(JsonRPC)DC_QR_ASK_VERIFYGROUPandDC_QR_ASK_VERIFYCONTACT, just that the dialog shown after scanning uses the translated stringqrscan_ask_join_channelAdd membershould be hidden, and insteadQR Invite Codeshould be shownchat_new_channel_hintshould be shown.TODO:
jnifolder)