Skip to content

Commit

Permalink
Changes 7 day forecast to 5 day forecast due to new API restrictions
Browse files Browse the repository at this point in the history
  • Loading branch information
elewin committed Jun 5, 2023
1 parent 3a0bb58 commit da05477
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion client/dist/bundle.min.js

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion client/src/AppContext.js
Original file line number Diff line number Diff line change
Expand Up @@ -374,7 +374,7 @@ export function AppContextProvider({ children }) {
].join("%2c");

const endTime = new Date(
new Date().getTime() + 6 * 60 * 60 * 24 * 1000
new Date().getTime() + 4 * 60 * 60 * 24 * 1000
).toISOString();

return new Promise((resolve, reject) => {
Expand Down
4 changes: 2 additions & 2 deletions client/src/components/weatherCharts/DailyChart/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ const createChartOptions = ({
stacked: false,
title: {
display: true,
text: `7 Day ${
text: `5 Day ${
altMode
? `Wind Speed / Precipitation (${lengthUnit})`
: `Temp / Precipitation`
Expand Down Expand Up @@ -218,7 +218,7 @@ const DailyChart = () => {
styles.errContainer
}`}
>
<div>Cannot get 7 day weather forecast</div>
<div>Cannot get 5 day weather forecast</div>
<div className={styles.message}>{dailyWeatherDataErrMsg}</div>
</div>
);
Expand Down

0 comments on commit da05477

Please sign in to comment.