search

skalnik / aqi-wtf

😷 WTF is the AQI near me right now?!
https://aqi.wtf
MIT License
44 stars 12 forks source link

Update app.js #43

Closed ramatevish closed 3 years ago

ramatevish commented 3 years ago

An exception is thrown when calling loadSensorsFromCacheAndShowAQI if no sensor data is stored locally.

vercel[bot] commented 3 years ago

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/skalnik/aqi-wtf/o6dfw0qeu
✅ Preview: https://aqi-wtf-git-fork-ramatevish-patch-1.skalnik.vercel.app

obra commented 3 years ago

LGTM. (@ramatevish - Just to confirm, this is a correctness fix, but the live site isn't failing to work for end-users, right?)

skalnik commented 3 years ago

Seems straight forward enough.

ramatevish commented 3 years ago

Correct.

On Tue, Sep 8, 2020 at 11:16 AM Jesse Vincent notifications@github.com wrote:

LGTM. (@ramatevish https://github.com/ramatevish - Just to confirm, this is a correctness fix, but the live site isn't failing to work for end-users, right?)

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/skalnik/aqi-wtf/pull/43#issuecomment-689051001, or unsubscribe https://github.com/notifications/unsubscribe-auth/AALTMVKD3YHCE35SEPM5ROLSEZYIHANCNFSM4RAKKGNQ .