search

sidlatau / flutter_email_sender

Allows send emails from flutter using native platform functionality.
Apache License 2.0
151 stars 84 forks source link

Added reponse for email client not exists case #36

Closed RohitKumarMishra closed 4 years ago

sidlatau commented 4 years ago

I think original behaviour is more universal - package client should handle errors from the plugin. For example, maybe other than the English language should be shown in the app - in that case, it is more convenient just to handle exception locally in the app.

RohitKumarMishra commented 4 years ago

We are not getting any exceptions or any messages from plugin, the plugin returns void, that is why i made the changes.

On Sat, 25 Jan, 2020, 3:50 PM Tautvydas Šidlauskas, < notifications@github.com> wrote:

I think original behaviour is more universal - package client should handle errors from the plugin. For example, maybe other than the English language should be shown in the app - in that case, it is more convenient just to handle exception locally in the app.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/sidlatau/flutter_email_sender/pull/36?email_source=notifications&email_token=ADM4CMEISSFOO2IQSLBVMMDQ7QG7TA5CNFSM4KKAEMHKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEJ4ZSLQ#issuecomment-578394414, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADM4CMAB3GMCCKDIXCFGODDQ7QG7TANCNFSM4KKAEMHA .