#20-aggregated-temp-for-month-year #267

Closed
b24hugst wants to merge 0 commits from #20-aggregated-temp-for-month-year into temp-week-2-merge-branch
Collaborator

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.

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.
returns average temperature for the written year
Shows 2026 average temperature currently
Showcases average temperature from that month
Outputs average temperature for written month
months addressed with ints instead of strings and also datatype int added in parameter
Decided it's better to save this for another issue
Noticed I missed this
needed to save database results into cache
Cache will now get checked before database, and updated every 10 minutes
returns average temperature from month, from year
added frontend communication to showcase monthly average temperature
Converts month number to month name at output
Add so card is shown outputing that there's no data for that month or year
/year now returns 404 instead of 500 when having no data
Collaborator

Assigned @c24carso to fix merge conflict

Assigned @c24carso to fix merge conflict
Collaborator

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.

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.
b24johka changed target branch from team_1_week_1 to temp-week-2-merge-branch 2026-04-15 09:17:06 +00:00
Collaborator

Manually merged

Manually merged
a24timsv closed this pull request 2026-04-15 10:38:00 +00:00

Pull request closed

Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
5 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Andras/BoundlessFlowCampus2K!267
No description provided.