Permission problem about nativescript-email

[NicoleZA]

Ok. I think I have found the problem. It seems to work only if you have an attachment.
So, to fix the problem I moved

  mail.setType("message/rfc822");
  mail.setAction(android.content.Intent.ACTION_SEND_MULTIPLE);

out of the if (arg.attachments) {.... section in email.android.js and it now works with or without attachments.

[EddyVerbruggen]

Hey @NicoleZA thanks for reporting this!

Since I don't have an Android 6 device... can you do me a favor and check if it would also work if you reverted your local changes and only commented out this line? I don't think that should be there.

Thanks!
Eddy

[NicoleZA]

OK. Sure. I have left the office, but will test it Monday and let you know

On 19 Aug 2016 2:39 PM, "Eddy Verbruggen" [email protected] wrote:

Hey @NicoleZA https://github.com/NicoleZA thanks for reporting this?

Since I don't have an Android 6 device... can you do me a favor and check
if it would also work if you reverted your local changes and only commented
out this line
https://github.com/EddyVerbruggen/nativescript-email/blob/master/email.android.js#L57?
I don't think that should be there.

Thanks!
Eddy

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#4 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGPYvc1WU02xTskspHeKSKRvKHXKTFvUks5qhaQWgaJpZM4JoWHJ
.

[EddyVerbruggen]

👍 Thanks in advance!

[NicoleZA]

I tried what you suggested, but it didn't work.
however
I did delete that line but also still moved this code above the if (arg.attachments) { line
mail.setType("message/rfc822"); mail.setAction(android.content.Intent.ACTION_SEND_MULTIPLE);
and this works

[EddyVerbruggen]

Thanks Nicole, really appreciate you testing this. I'm releasing 1.3.1 with your suggested fix.