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

Restrict email with attachment to just mail apps on android #52

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

Conversation

devxpy
Copy link

@devxpy devxpy commented Jul 22, 2022

Fixes #49

@github-actions github-actions bot added the Android android related issues label Jul 22, 2022
@taljacobson
Copy link
Owner

sorry for the late response.
tried your branch out with the example app and was not able to launch any email client's with attachments with your changes.

@devxpy
Copy link
Author

devxpy commented Aug 21, 2022

Thanks for the response. Will try out the example app to figure out what's going wrong there. (Had been testing on my app directly.)

@VladShturma
Copy link

@taljacobson example app starts working if you add the needed code to your AndroidManifest.xml

<application .... />
// add queries tag for mailto intent out side of application tag
<queries>
  <intent>
    <action android:name="android.intent.action.SENDTO" />
    <data android:scheme="mailto" />
  </intent>
</queries>

I believe you should update package's setup instruction and add this code there.

Because there was an issue, that you advised to fix the same way.
#38 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Android android related issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Sharing Intent option mail app as first
3 participants