Noticed an issue right before the end of the day with my code in regard to the forecast window code. Sometimes doesn't send any requests at all for example when changing from 4 to 5, but this is very inconsistent for me, sometimes it works, sometimes it doesn't, this only seems to be an issue on the summary page, however. It works great on the single city page.
If we select a forecast base window that we don't have a time available for then ideally there should be no requests fired.
If we select forecast window of 5, then we should still be able to select smaller ranges.
Actual Behaviour:
Even if we only have a base time of 1 available. We can still click 2-3-4 and 5 and have them fire requests.
AC:
Only fetch past data within the forecast window
Check to see if this happens for you. (Might just be a me issue)
If it is an issue for you, fix bug
On a hard refresh, user is unable to click lower forecast window times besides 1.
Note that this does not happen for dates that are able to fully encompass a 5-day dataset. For example, if we try on the 14/08/2024 we will not see this. Plus, the forecast time range also updates as expected.
Noticed an issue right before the end of the day with my code in regard to the forecast window code. Sometimes doesn't send any requests at all for example when changing from 4 to 5, but this is very inconsistent for me, sometimes it works, sometimes it doesn't, this only seems to be an issue on the summary page, however. It works great on the single city page.
Steps to reproduce:
https://github.com/user-attachments/assets/ebf27721-1d5f-4cef-832c-514040b388e3
Expected behaviour:
Actual Behaviour:
AC:
On a hard refresh, user is unable to click lower forecast window times besides 1. Note that this does not happen for dates that are able to fully encompass a 5-day dataset. For example, if we try on the 14/08/2024 we will not see this. Plus, the forecast time range also updates as expected.