katzer / cordova-plugin-email-composer

Edit and send email messages
Apache License 2.0
344 stars 333 forks source link

fix Android 11 email intent params & targets #365

Closed PetePrattis closed 2 years ago

PetePrattis commented 2 years ago

Fixes Android 11 target sdk 30 crash, fixes email intent params and now can successfully attach multiple files, pop up menu shows the correct target apps.

jfoclpf commented 2 years ago

@katzer do you know when and if you will publish these changes to npm?

jfoclpf commented 2 years ago

@PetePrattis I have the same problem, it crashes in Android 11, how did you manage this? Why did you close the PR?

PetePrattis commented 2 years ago

@jfoclpf Well my pr was a workaround, this pr #368 seems to be the actual solution, worked for me.

jfoclpf commented 2 years ago

@PetePrattis could you kindly tell me in #361 what is exactly the solution? Thanks

jfoclpf commented 2 years ago

@PetePrattis do you mean PR #368 is the stable solution?

PetePrattis commented 2 years ago

@jfoclpf Yes sorry, I meant #368, the one that I use for my production. Which I have tested on multiple devices, target sdk 30, Xiaomi devices and Samsung with android 10 &11.

jfoclpf commented 2 years ago

@PetePrattis can I ask you exactly how did you implement it in production since this project seems to be abandoned?

Did you fork it and fetched from your own repo?