Conversation
|
Thanks for the PR. Looks good overall. Will merge soon. |
| } | ||
| } | ||
| } | ||
| } |
There was a problem hiding this comment.
This is a syntax error. One too many closing brackets.
README.md
Outdated
| body: '<b>A Bold Body</b>', | ||
| isHTML: true, | ||
| attachment: { | ||
| attachments: {[ |
There was a problem hiding this comment.
Need to switch the order of the curly and square brackets.
There was a problem hiding this comment.
Thanks will sort now..
|
when will this be merged? Need the ability to add multiple attachments |
|
Good question. @chirag04? For now you can use my fork https://github.com/adamski/react-native-mail |
|
How has this not still been added.. really disappointing. |
|
Woah! Can't believe this is still not possible ... Please Merge! |
|
What’s the latest with getting support for this merged into a release? Or is there a better fork we all should switch to? |
|
Can folks try the latest version and see if this is still an issue? |
Rebased #11 (the first PR for multiple attachments). Tested and working on iOS, and with Android changes from @kneza23
Breaking changes
attachment:{}is nowattachments:[], e.g.: