-
Notifications
You must be signed in to change notification settings - Fork 0
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
Push UAV forecast to farmcalendar #35
Conversation
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.
Only two small comments, but neither are a major thing, more like a footnote on the mind for future improvements.
One is the numericValue that is not longer in the observation.
The other comment is that it might be helpful to have the endpoints for FC as config env vars (with a default value set). This probably will make the transition to using the gatekeeper easier in the future.
… forecast api calls
f68089e
to
d0a01c8
Compare
Change
Small changes
create_point
->find_or_create_point
numericValue
instead ofhasValue
#37Relates #29
Adresses #37 #34