Error handling about ngupload HOT 6 CLOSED

Good Point, I would like to start an unstable branch and implement promises, then we could handle errors with the promise api. want to help? or have or suggestions?

from ngupload.

Personally I would take the lazy way out and just throw that line inside of a try/catch(err), with a return of the error. I don't know much about the JS promise API, but at the moment the support looks very low:

http://caniuse.com/promises

from ngupload.

OK, I added error handling. See this Gist:

https://gist.github.com/djvs/7c82f7395624d85c7e03

specifically lines 96 and 185-219 (one big try-catch). This adds support for an optional error-catcher attribute in the <form> tag. You may want to double check, I basically used your method for the ng-upload attribute.

from ngupload.

This would be huge for me. +1

from ngupload.

I will try to incorporate it by the weekend, I have been so busy, if you
need it before, wrapping it up in a pull-request would be welcomed.

Thanks

Tom

On Thu, Jul 31, 2014 at 1:49 PM, Ken Caron [email protected] wrote:

This would be huge for me.

—
Reply to this email directly or view it on GitHub
#112 (comment).

Tom Wilson
Jack Russell Software Company Division of CareKinesis
494 Wando Park Blvd
Mount Pleasant, SC 29464
Phone: 843-606-6484
Mobile: 843-469-5856
Email: [email protected]
Web: http://www.jackhq.com
Calendar:
http://www.google.com/calendar/embed?src=tom%40jackrussellsoftware.com&ctz=America/New_York
http://www.jackhq.com/calendar

This e-mail may contain information that is confidential, privileged or
otherwise protected from disclosure by the Health Insurance Portability and
Accountability Act (HIPAA) and other state and federal laws. This
information is intended only for the individual names above. Any review,
use disclosure or dissemination of this material is strictly prohibited.
If you receive this information in error, please notify CareKinesis
immediately at 888-974-2763 and delete the original at once.

from ngupload.

Added to master js, but not released...

Let me know if you see any issues:

from ngupload.