hepcat72 / node-red-contrib-life360

GNU General Public License v3.0
3 stars 4 forks source link

Correctly handle errors from server #6

Closed brianmay closed 2 years ago

brianmay commented 2 years ago

Previously we would treat errors as non-errors and then crash trying to parse the error data.

brianmay commented 2 years ago

As referenced in #1.

hepcat72 commented 2 years ago

The merge button never loads on my phone or I'd merge it right now.

brianmay commented 2 years ago

I fixed the conflicts.

hepcat72 commented 2 years ago

I assumed you had with that last push. I'll merge when I'm not out on my phone, because all GitHub gives me is a spinning circle instead of a status and merge option:

image

MaksimShakavin commented 2 years ago

Will you also push an update to npm?

hepcat72 commented 2 years ago

Yes. Not sure when though. You can install via github however...