# Please review my Local Weather app

**URL:** <https://forum.freecodecamp.org/t/please-review-my-local-weather-app/31880>\
**Category:** I Built This\
**Created:** [August 28, 2016, 2:33pm UTC](https://forum.freecodecamp.org/t/please-review-my-local-weather-app/31880 "2016-08-28T14:33:05Z")\
**Posts on this page:** 8\
**Page:** 1

<div class="post-metadata">

**Author:** ![hepabolu](https://sea3.discourse-cdn.com/freecodecamp/user_avatar/forum.freecodecamp.org/hepabolu/32/164694_2.png) [@hepabolu](https://forum.freecodecamp.org/u/hepabolu)\
**Post date:** [August 28, 2016, 2:33pm UTC](https://forum.freecodecamp.org/t/please-review-my-local-weather-app/31880/1 "2016-08-28T14:33:05Z")

</div>

[My take on the Local Weather app challenge](http://codepen.io/hepabolu/full/bZzYXX/)

I’d love to hear feedback

---

<div class="post-metadata">

**Author:** ![jscogg](https://sea3.discourse-cdn.com/freecodecamp/user_avatar/forum.freecodecamp.org/jscogg/32/165014_2.png) [@jscogg](https://forum.freecodecamp.org/u/jscogg)\
**Post date:** [September 9, 2016, 10:07pm UTC](https://forum.freecodecamp.org/t/please-review-my-local-weather-app/31880/2 "2016-09-09T22:07:48Z")

</div>

Not working for me. 🙁

---

<div class="post-metadata">

**Author:** ![hepabolu](https://sea3.discourse-cdn.com/freecodecamp/user_avatar/forum.freecodecamp.org/hepabolu/32/164694_2.png) [@hepabolu](https://forum.freecodecamp.org/u/hepabolu)\
**Post date:** [September 10, 2016, 9:59am UTC](https://forum.freecodecamp.org/t/please-review-my-local-weather-app/31880/3 "2016-09-10T09:59:07Z")

</div>

Can you be more specific? What is not working?

---

<div class="post-metadata">

**Author:** ![jscogg](https://sea3.discourse-cdn.com/freecodecamp/user_avatar/forum.freecodecamp.org/jscogg/32/165014_2.png) [@jscogg](https://forum.freecodecamp.org/u/jscogg)\
**Post date:** [September 11, 2016, 3:59am UTC](https://forum.freecodecamp.org/t/please-review-my-local-weather-app/31880/4 "2016-09-11T03:59:30Z")

</div>

Well, I may have spoken too soon, but still this could be improved upon I suppose…

 ![](https://global.discourse-cdn.com/freecodecamp/original/3X/4/5/45a4db203627cedb43e1595c8bdd9a0556f89016.png)

This is all a user sees when first loading up the page. At this point, the app appears to be broken or in a non-functional state. It is not until a user clicks on either the Fahrenheit or Celsius buttons that any content is loaded. So, from a UX perspective, you might think about instead of showing a page with unloaded content you might instead show a page with perhaps simply a button to click first to then show the retrieved content based on whether or not they prefer a specific unit of measurement.

Also, I noticed that even if you click on one of those buttons, the temperature is not converted. See:

![](https://global.discourse-cdn.com/freecodecamp/original/3X/6/2/62a708a8a1e27c4c64a0633670d90df8e1dab21e.png)

Additionally, the weather is not accurate to the user’s location. For example, I live in Seattle, WA and this is what I’m shown:

 ![](https://global.discourse-cdn.com/freecodecamp/original/3X/7/0/70647ad641913f3cdfb2bb992774f6037bbd7ea3.png)

Outside of that it seems I was a bit quick off the line. Overall, you’ve used Bootstrap and the API to present the information in a pleasing way. 🙂

---

<div class="post-metadata">

**Author:** ![hepabolu](https://sea3.discourse-cdn.com/freecodecamp/user_avatar/forum.freecodecamp.org/hepabolu/32/164694_2.png) [@hepabolu](https://forum.freecodecamp.org/u/hepabolu)\
**Post date:** [September 11, 2016, 5:44am UTC](https://forum.freecodecamp.org/t/please-review-my-local-weather-app/31880/5 "2016-09-11T05:44:45Z")

</div>

Ah. I think I know the root of the problem: it’s unable to figure out your location. I’ve defaulted to Amsterdam, just in case, but I could never test that.

You’re right I could do a better job in that situation, so I’m going to try to improve that. But I’m still curious why it can’t find your location. I’ve struggled some time with the method used in the course. I could never get it to work, so I’ve switched to using a website: [http://ip-api.com/json](http://ip-api.com/json). Could you check if that website returns data that matches your location?  
.

---

<div class="post-metadata">

**Author:** ![jscogg](https://sea3.discourse-cdn.com/freecodecamp/user_avatar/forum.freecodecamp.org/jscogg/32/165014_2.png) [@jscogg](https://forum.freecodecamp.org/u/jscogg)\
**Post date:** [September 11, 2016, 7:17am UTC](https://forum.freecodecamp.org/t/please-review-my-local-weather-app/31880/6 "2016-09-11T07:17:25Z")

</div>

It does! It’s off a little as to be expected because it’s using my IP, but it’s a heck of a lot closer than Amsterdam. 😛

---

<div class="post-metadata">

**Author:** ![hepabolu](https://sea3.discourse-cdn.com/freecodecamp/user_avatar/forum.freecodecamp.org/hepabolu/32/164694_2.png) [@hepabolu](https://forum.freecodecamp.org/u/hepabolu)\
**Post date:** [September 11, 2016, 7:24am UTC](https://forum.freecodecamp.org/t/please-review-my-local-weather-app/31880/7 "2016-09-11T07:24:28Z")

</div>

I’ve updated the Codepen and it should give a warning when it cannot find your location. So that situation should be handled better.

Can you check that?

---

<div class="post-metadata">

**Author:** ![ILM](https://sea3.discourse-cdn.com/freecodecamp/user_avatar/forum.freecodecamp.org/ilm/32/363239_2.png) [@ILM](https://forum.freecodecamp.org/u/ILM)\
**Post date:** [June 1, 2021, 6:07pm UTC](https://forum.freecodecamp.org/t/please-review-my-local-weather-app/31880/8 "2021-06-01T18:07:58Z")

</div>


