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

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in 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