-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
EmailComposer issue #168
Comments
function now returns a promise that resolves if email composer is available, and rejects if it's not partially fixes #168
Hey @Langstra Thanks for pointing out these issues. I applied some changes and will go ahead and test them now. The Note that after the recent changes are published, you have to import it from |
@Langstra I just tested the plugin on my Android Lolipop device. The wrappers are working correctly now. However, the plugin itself isn't functioning properly (at least on my device?) |
I will test it as well and if necessary create an issue at the page of the plugin itself. Thank for looking into it so quickly. |
By looking at the source of emailcomposer.js and looking at
it does not return anything, correct? |
@ihadeed I tested the wrapper some more and I found that the isAvailable() function gives an error when calling it. The error is because of the plugin itself you think?
The open function does not appear to be called at all. I inserted console.log() at the beginning of the function in email-composer.js, but it never appeared in my console. |
@Langstra I'm going to open an issue with the developer and see if they can alter the implementation and fix the issues there. |
There are four issues I found when using the EmailComposer plugin.
I created a demo to show the issue:
https://github.com/Langstra/ionicNativeDemo
The text was updated successfully, but these errors were encountered: