search

dhruvabhat24 / weather-app

It is a weather app made using react.js
https://weather-app-lac-pi.vercel.app
MIT License
17 stars 21 forks source link

Fetch Report is kinda useless #50

Open fancyvanilla opened 6 months ago

fancyvanilla commented 6 months ago

The "Fetch Report" button simply updates the forecast data without displaying anything to the user, whereas the "Download Report" button updates the forecast and allows for downloading. I suggest two potential solutions:

Add feedback to the "Fetch Report" action, such as displaying a message saying "Data fetched correctly, you can now download." Additionally, before enabling the "Download Report" button, verify if the data has been successfully fetched.

Alternatively, consider removing the "Fetch Report" button entirely if its function is redundant due to the "Download Report" action already updating the forecast.

These options would enhance the user experience by providing clarity and ensuring that users are aware when the data has been updated and is ready for download.

dhruvabhat24 commented 6 months ago

That's a great idea. I can assign this to you if you are intrested to work on it

fancyvanilla commented 6 months ago

yes i will work on it and hopefully the download bug gets fixed too

dhruvabhat24 commented 6 months ago

I have assigned this to you you can work on this