Skip to content
New issue

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

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

Already on GitHub? # to your account

Fixed sharing to Snikket #56

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Fixed sharing to Snikket #56

wants to merge 1 commit into from

Conversation

mwild1
Copy link
Member

@mwild1 mwild1 commented Aug 22, 2021

Fixes #18

@mwild1
Copy link
Member Author

mwild1 commented Aug 22, 2021

@khaliddd I noticed two problems while testing. I shared a screenshot with a contact and typed "Test share" in the share dialog.

  1. The sent messages are sent unencrypted, although I have encryption enabled for this contact.
  2. When I view the conversation in the app, the sent text is visible, but not the shared image.

Note: If either of these would take significant time to fix, please prioritize #52 instead for now and we can come back to this one later.

Screenshots:

User (iOS) Contact (Android)
snikket-ios-pr-56-user snikket-ios-pr-56-contact

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unable to share attachments from other applications
3 participants