search

capacitor-community / http

Community plugin for native HTTP
MIT License
209 stars 136 forks source link

fix(ios): Add missing request error handling #225

Open mrahn24 opened 2 years ago

mrahn24 commented 2 years ago

resolves #224

jwktje commented 2 years ago

When might this be merged?

tomsawdayee commented 2 years ago

Any update on this? Please merge this as this is very basic functionality that is missing. Thanks

flipace commented 2 years ago

Just a hint, for others who might be waiting on this as well -> We use patch-package to apply the changes of this (and other) PRs on our builds without requiring us to fork.

In such cases where there is a PR already open, you can simply append .patch to the github url to get the patch code: https://github.com/capacitor-community/http/pull/225.patch

This you can put into ./patches/@capacitor-community+http+1.4.1.patch of your project and when you have patch-package installed, it will auto-apply the patch to the package in node_modules whenever you update/install.

tomsawdayee commented 2 years ago

Wow! Thanks a bunch @flipace ! A perfect workaround for me. I almost went back to old cordova plugins!