#20-aggregated-temp-for-month-year #267
No reviewers
Labels
No labels
_CRITICAL_
API
app
backEnd
Blocked, waiting for further changes
bug
cleanup
close
design
duplicate
enhancement
feature request
frontEnd
help wanted
invalid
low priority
needs input
needs review
project documentation
question
research
reviewed
script
security
SQL
style
testing
topLevel
wontfix
No milestone
No project
No assignees
5 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
Andras/BoundlessFlowCampus2K!267
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "#20-aggregated-temp-for-month-year"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
New functions to output average temperature from /year and /year/month. Works with a memory cache to not recalculate each request.
If chosen month or year has no data it will say so in a card.
This is part of the #20 issue, where /year/week is yet to be implemented. I chose to do a pull request now before the weekly merge and start on the /year/week afterwards with no stress since
the weekly implementation seemed a little bit more tricky than year and month.
Assigned @c24carso to fix merge conflict
The code executes as intended and nothing breaks. Two views are shown in the frontend: one with year and one with month and year. Both currently display the text “NO DATA”.
The code follows established coding standards. Naming conventions and spacing look good.
There is a lack of comments, which would be helpful for a quick overview of the code.
Manually merged
Pull request closed