search

alexis-jaksone / forecast-plus

Provides local and long range weather forecast right in your browser toolbar
http://add0n.com/forecast-plus.html
19 stars 4 forks source link

Badge reports NaN instead of temperature #16

Open braiam opened 8 years ago

braiam commented 8 years ago

I'm not sure what should be the desirable behavior. We validate val before consume it, to make sure that it is a number and then either: show an error (something like "!" in red?) or just reuse the old one.

I'm not sure what triggered this, and couldn't see from where val comes from, but I think that we should validate data before using it anyways.

alexis-jaksone commented 8 years ago

Can you describe steps to regenerate the NaN on badge?

braiam commented 8 years ago

I'm not sure what triggered this

But I would presume that it was due a timeout.