-
Notifications
You must be signed in to change notification settings - Fork 12
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
New API #85
New API #85
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
This now works, but due to how the API is paid for now, each sensor and field returned is counted against you. I need to adjust this to get a bounding box of your area for the sensors and maybe bust the cache if you're further than like 5km from one or something. |
https://github.com/davidwood/node-geopoint/blob/master/geopoint.js#L123-L159 can maybe borrow this MIT open source impl |
I ended up using ChatGPT for this |
070e30d
to
f9897fe
Compare
@@ -37,13 +42,18 @@ | |||
if (sensor_data == null) { | |||
throw "Sensor data not cached"; | |||
} | |||
if (sensor_data.version !== 1) { | |||
if (sensor_data.version !== 2) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wonder how long this has been busted 💀
Imma leave this up in preview for a day so I can see how much my testing cost in terms of tokens and then merge it |
To be fair, they did email me, but I also forgot.
Closes #84