Let's discuss your "Local Weather App"

Code Feedback
1

Greetings, Fellow Campers!

This topic is for listing and review of Local Weather App projects built as a part of FCC challenges. Please stick to these basic rules to keep the topic clean:-

  • Post your Codepen (or other) link to your project as a general reply to this topic.
  • For Codepen links use the following format to avoid embedding the pen:-
Project Link - [http://codepen.io/santakmishra/full/pgqOYq/](http://codepen.io/santakmishra/full/pgqOYq/)
  • Post reviews as a specific reply to the link which you are reviewing.
1 Like
2

Project Link - http://codepen.io/santakmishra/full/yemxgm

5 Likes
3

Hi, looks nice. I wonder, am I the only one having problems with the cross-origin thing? Because this pen only worked by using crossorigin.me. I had the same with my Random Quote Machine.

4

JSONP is another way to go, but only of the API server supports it. Open Weather API does support JSONP, so I went with that. Otherwise CORS proxies seem to be the only way to go.

1 Like
5

Project Link - http://codepen.io/RickStewart/full/yOYpjv/

9 Likes
6

Project Link - http://codepen.io/annamyself/full/aNxjdq/

7

I loved this project, I learnt quite a lot of stuff !
@santakmishra I was disgusted by the OpenWeather icons they were so UGLY, so I looked up where you got yours, and decided to use yours. (I am actually pointing directly to your server, so if you mind tell me and I can remove stop using it)

Project Link - http://codepen.io/soueuls/pen/LZYmVj/

2 Likes
8

Your project looks elegant and no issues at all with pointing to my server for icons. Nice touch with the Hello little human… :slight_smile: Except, I didn’t find the C to F temperature convertor function anywhere. Maybe I missed it?

I must inform you though that I am planning to upload all my project assets to Codepen itself once I start using the Pro version. I might remove the assets from my server after that. Won’t happen for the next few months. But, I suggest you find a more permanent solution for future.

9

Finished this up today. I didn’t spend a lot of time on the design aspects of the page. I may go back soon and redo the background of the page, as well as some of the other design elements. But, I was really happy with the functionality code and wanted to keep working through some other projects.

Project Link - http://codepen.io/bsandusky/full/oLgyNM/

1 Like
10

Here’s mine. I did it a while ago. I used IP-API to get the location.
It was fun to do.

Project Link - [http://codepen.io/Topzie/full/QNaZJv/] (http://codepen.io/Topzie/full/QNaZJv/)

3 Likes
11

This one was pretty fun, and I had to learn how callbacks work to finish it up.

Project Link - https://codepen.io/imtoobose/full/Pzqbxq/

10 Likes
12

Hi all. I’m really enjoying making this project, but am having some troubles with it. I’m not really sure what I’m doing wrong and I’ve tried reading up around the subject and I’m still not wiser. I’d really appreciate any suggestions anyone has on what is wrong with the background - it doesn’t change based on the weather ID.

Project link is: http://codepen.io/andyst81/pen/JKdMmN

Thank you.

2 Likes
13

Here is mine : http://codepen.io/rameshsyn/full/zrpXbg

14

Okay I’m really stuck on this one. navigator.geolocation.getCurrentPosition seems to only work when using https but when I switch codepen.io to https openweathermap fails. I know others have had this problem, but I can’t find a working solution other than the demo. Can anyone point me in the right direction? I’ve looked over a number of other people’s submissions but none of them seem to work :confused:

1 Like
15

Hey, I had the same problem. You can use crossorigin.me to overcome that: instead of http://openweathermap.org/whatever you can use https://crossorigin.me/http://openweathermap.org/whatever. Or alternatively, you can use some kind of api that gets the location based on ip address.

9 Likes
16

Thanks for the tip. I gave it a try and it still failed. Went to crossorigin.me and it said the server is down :expressionless: When I get off work tonight I will try again and probably try to find a more reliable cors server. Anyway thanks again.

17

The website is indeed regularly down. Let me now if you find another site.

18

Well, when I got home the crossorigin.me server was up again so I stuck with that and managed to get some more work done on my projects. However about 3 hours later it was down again, so I went looking for an alternative and found https://cors-anywhere.herokuapp.com/ . It seems to be stable and is working fine for me. They do limit request to 600 per hour, but for the purpose of this project that is more than enough.

10 Likes
19

Project Link -https://codepen.io/Josh5231/full/BzNgwW/
You will need to be in HTTPS for this to work. PLease let me know what you guys think.

1 Like
20

Here’s mine. Just finished it.

Project Link - http://codepen.io/RadEdje/full/ZOQReG/

I found a cool jquery plugin for the weather (using yahoo weather api)… it’s in the link of references at the bottom of my pen if anybody wants to use it. The developer has a great you tube tutorial. Then I realized i was losing out on learning about openWeatherMap api’s so I made a second weather app machine… yahoo and openWeatherMap don’t seem to agree very much though. I put the 2 side by side.

Any constructive criticism would be great. Thanks.

By the way,
If any body has trouble with the location api… this thread in our forum was really helpful. I’d like to thank everyone on that thread too for all their help and advice.

1 Like