For iOS, don't return the feedback MFMailComposeResult result

[herveperoteau]

Hi,

For iOS, in the MFMailComposeViewControllerDelegate, you don't return the feedback MFMailComposeResult result.
It could be easily by keep the result:(FlutterResult)result in a variable and call it in the MFMailComposeViewControllerDelegate method.

[taljacobson]

yes there is a pending PR #28 i made to return the result.

i have not been able to properly test it though due to issues with my physical device, i did make it available on pub with a dev version of 0.6.0-dev.1.

hopefully i will be able to test and merge it soon.

[herveperoteau]

Ok I could test it on device What is the correct syntax to use in my yaml to use your branch ? # flutter_mailer: ^0.4.1+1 flutter_mailer: https://github.com/taljacobson/flutter_mailer/tree/MailerResponse

…

On 30 Jul 2020, at 13:15, Tal Jacobson ***@***.***> wrote: yes there is a pending PR #28 <#28> i made to return the result. i have not been able to properly test it though due to issues with my physical device, i did make it available on pub with a dev version of 0.6.0-dev.1. hopefully i will be able to test and merge it soon. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#31 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAXAWXCEWJSSENBNT2IBXQDR6FI3PANCNFSM4PNO3TMA>.

[herveperoteau]

Missing } in your branch

…

On 30 Jul 2020, at 13:15, Tal Jacobson ***@***.***> wrote: yes there is a pending PR #28 <#28> i made to return the result. i have not been able to properly test it though due to issues with my physical device, i did make it available on pub with a dev version of 0.6.0-dev.1. hopefully i will be able to test and merge it soon. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#31 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAXAWXCEWJSSENBNT2IBXQDR6FI3PANCNFSM4PNO3TMA>.

[herveperoteau]

After the fix of missing } it’s works as expected on iOS device. (See my screenshot) Did you know when you could published it ? Because I need this feature quickly. Please give me a feedback Thanks Hervé

…

On 30 Jul 2020, at 13:30, herve peroteau ***@***.***> wrote: Missing } in your branch <Screenshot 2020-07-30 at 13.29.16.png> > On 30 Jul 2020, at 13:15, Tal Jacobson ***@***.*** ***@***.***>> wrote: > > > yes there is a pending PR #28 <#28> i made to return the result. > > i have not been able to properly test it though due to issues with my physical device, i did make it available on pub with a dev version of 0.6.0-dev.1. > > hopefully i will be able to test and merge it soon. > > — > You are receiving this because you authored the thread. > Reply to this email directly, view it on GitHub <#31 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAXAWXCEWJSSENBNT2IBXQDR6FI3PANCNFSM4PNO3TMA>. >

[taljacobson]

ok published a new dev version 0.6.0-dev.2, please try it out,

[herveperoteau]

It works well Thanks

…

On 30 Jul 2020, at 14:35, Tal Jacobson ***@***.***> wrote: ok published a new dev version 0.6.0-dev.2, please try it out, — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#31 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAXAWXGYMPB64LIBXXFFC4DR6FSIXANCNFSM4PNO3TMA>.