search

dclaar / purple_air

Code for displaying Air Quality from a Purple Air device
MIT License
6 stars 4 forks source link

Broken by new Purple Air API #17

Closed danswett closed 1 year ago

danswett commented 2 years ago

Alas the new purple air API change has broken this wonderful integration.

New API requires an API key.

Will try to find time to poke at this myself, but wanted to raise the issue as well since both of my M5Sticks are now broken.

dclaar commented 2 years ago

You will need to email purple air for your own API key.

I will upload a fix shortly--O(days), not O(hours)--just need to update the docs a bit, make sure tests still pass.

danswett commented 2 years ago

Thank you!!

On Wed, Jun 15, 2022 at 8:32 PM Doug Claar @.***> wrote:

You will need to email purple air for your own API key.

I will upload a fix shortly--O(days), not O(hours)--just need to update the docs a bit, make sure tests still pass.

— Reply to this email directly, view it on GitHub https://github.com/dclaar/purple_air/issues/17#issuecomment-1157193623, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA4JWNNZIDHRLXYWLWT3BX3VPKN27ANCNFSM5YS3FADQ . You are receiving this because you authored the thread.Message ID: @.***>

dclaar commented 2 years ago

As noted, you will need an API read key, but should be fixed!

Files changed:

 README.md            | 30 ++++++++++++++++++++++--------
 copy_for_web.py      | 23 ++++++++++++++---------
 copy_to_stick.md     | 24 ++++++++++++++++--------
 flash/apps/WebAQI.py | 25 ++++++-------------------
 flash/aqi.py         |  6 ++++--
 flash/test_WebAQI.py | 23 +++++------------------
 6 files changed, 67 insertions(+), 64 deletions(-)

Really, you need to copy these new versions:

 flash/apps/WebAQI.py
 flash/aqi.py

And modify your aqi_web.json, adding the comma and the read_api_key bit:

{"name": 0, "correction_index": 2, "sensor_location": "38889", "brightness": 2, "read_api_key": "########-####-####-####-############"}

danswett commented 2 years ago

Thank you!!

On Sat, Jun 18, 2022 at 2:05 PM Doug Claar @.***> wrote:

As noted, you will need an API read key, but should be fixed!

Files changed:

README.md | 30 ++++++++++++++++++++++-------- copy_for_web.py | 23 ++++++++++++++--------- copy_to_stick.md | 24 ++++++++++++++++-------- flash/apps/WebAQI.py | 25 ++++++------------------- flash/aqi.py | 6 ++++-- flash/test_WebAQI.py | 23 +++++------------------ 6 files changed, 67 insertions(+), 64 deletions(-)

Really, you need to copy these new versions:

flash/apps/WebAQI.py flash/aqi.py

And modify your aqi_web.json, adding the comma and the read_api_key bit:

{"name": 0, "correction_index": 2, "sensor_location": "38889", "brightness": 2, "read_api_key": "########-####-####-####-############"}

— Reply to this email directly, view it on GitHub https://github.com/dclaar/purple_air/issues/17#issuecomment-1159562855, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA4JWNPSKHJDOPVKCUQGXCLVPY22NANCNFSM5YS3FADQ . You are receiving this because you authored the thread.Message ID: @.***>