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

Fixes for outlook #65

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

Fixes for outlook #65

wants to merge 1 commit into from

Conversation

kacyn
Copy link

@kacyn kacyn commented Jun 9, 2017

@chirag04

This PR includes fixes for Outlook. Outlook only gets attachments from ACTION_SEND and it also needs the output of Html.fromHtml(body) to return a string.

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@@ -63,7 +69,7 @@ public void mail(ReadableMap options, Callback callback) {
if (options.hasKey("body") && !options.isNull("body")) {
String body = options.getString("body");
if (options.hasKey("isHTML") && options.getBoolean("isHTML")) {
i.putExtra(Intent.EXTRA_TEXT, Html.fromHtml(body));
i.putExtra(Intent.EXTRA_TEXT, Html.fromHtml(body).toString());
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just curious, is this change necessary?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes - it doesn't work in Outlook otherwise.

@kacyn
Copy link
Author

kacyn commented Oct 17, 2017

@chirag04 Can we please get this merged?

@lchapoy
Copy link

lchapoy commented Mar 6, 2018

Facing the same issue on my project.

Could you please merge this MR.

MrEliasen added a commit to MrEliasen/react-native-mail that referenced this pull request Jan 14, 2019
@MaxPMagee
Copy link

MaxPMagee commented Jun 30, 2021

I've resolved the merge conflict in a new PR, but I really don't understand how/why this worked in the first place, so it could use some extra review. @kacyn if you have time, please let me know if this seems ok to you.
#183

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.

4 participants