Hi, just finished the weather app and looking for some feedback.
All comments appreciated, thanks for your time.
Its working but you don’t show location(city name/country)
1 Like
Thats right, that is actually not part of the brief.
The specific item is :
User Story: I can see the weather in my current location.
It doesnt actually say I need to see my location.
I found the api tended to give the incorrect location at times so I removed it and replaced it with wind speed and humidity.
Then how can I be sure as an user that the weather info I get is for my location?
I would like to say thats not the point, the point is…“did I satisfy the brief?”
The fact is the API provided by FCC is glitchy, this results in an unreliable result, I stand by my original reason to omit it.
But just to prove that I can do it, I did it.
Thanks for your feedback.
Edit: Turns out that the entire output is unreliable not just the location so including the location is valuable to the user to ensure he is getting the correct report or a report he should dismiss.
Leaving the location in my project for this reason, thanks again.