Let's discuss your "Local Weather App"

This was a very exciting challenge, I’ve learned a lot! Here is my version: https://codepen.io/Flok/full/yXgGwp/

I used freegeoip for location track and Dark Sky as the weather API. I’m also using Dark Sky’s icons to show the weather summary as an icon instead of background change. I’m not happy for that I had to use switch statement to add the related icons, but for some reason skycon’s js couldn’t recognize variables in its add function.

Any feedback is much appreciated.
Thanks o7

You can delete your switch statement with the skycon.add(…) with skycons.set(“icon”,response.currently.icon). Gets rid of the need for the switch.

The api you use for user location provides the city, why did you decide to just use the country name?

Also, for your darksky url you can just use callback=? instead of using crossorigin.me

Thanks Jebbss, the .set really helped me to clean up the switch mess.

In my location the API doesn’t provide the city any value, that was the reason why I decided to leave it out. I’ve changed the code so if there is a valid city data, it should now display it.

I’ve reworked the API call so it’s also cleaner now (I believe what I’ve done is somewhat a “promise”).

I’m also going to find out if I can sort out codepen’s access deny with the callback=?, not yet sure how. Thanks for the feedback!

https://codepen.io/prashant7196/full/rwmYgx/

Hello guys…This is my local weather app. please give me your valuable review

BRO ,if i was jury , i must give you an award for brilliant design…
awesome work… keep , doing great work…

in meantime , i would like to hear some word from your side on my project… here is the link

Dude, your project looks pretty good

I like that you kept it simple and focused on the main functionality of it. Great Job :clap:

Here’s mine https://codepen.io/Dee73/full/EXmXVY/

Had to unfortunately have an extra api call using google’s reverse geocode to get the city info, as the response from darksky didn’t have that. Overall quite a learning experience, with these projects I am just trying to reverse engineer the sample project without getting bogged down with excessive css/html/bootstrap, you can really spend a lot of time tinkering with that stuff, once I get the core functionality down I want to move on to the next project.

2 Likes

I decided to use the persons IP address as a basis for where the weather is located instead of relying on a location prompt from the browser. In the future Im going to add a button to let people put their zip code in and have it update based on that, and maybe a severe weather alert that shows up below box. I feel like I could keep going and going adding features and functions, but this will have to do for now. Feedback is always appreciated!

Thanks, this saved me from collapsing :smile:

So, I’m having trouble. I can pull the location data but I cant seem to get the dern thing to plug in the weather data. I’m thinking its an API/codepen issue. But it’d be great if someone could look it over for any other issues?

usually helps if I throw this in there huh?
https://codepen.io/DaBubbla/pen/OmmLNq?editors=0011

The first problem I see is that you are using jsonp as the data type, whereas it should be just json. The 2nd issue I see is the same problem I had: openweathermap doesn’t use https, codepen does, and it’ll cause a mixed-data error. Some people get around this by using some swapping service like originswap.me or herokuapp, but you run in to issues with too many requests, causing it to not work. Instead of using openweathermap, use darksky and you should be able to get it to work. If you’d like a reference, I used ipapi and darksky in my build a couple posts above.

Hi, A lot of us have “geolocation” disabled on our browsers… So if the GeoLocation call fails I coded it to ask the user for a Zip code (Which makes it US centric I realize…)
Project link - https://codepen.io/AzFalconer/full/WOjzJY/

Finally finished my weather app. Not as difficult as I thought it was before certain issues were resolved.
Issues encountered by me was that my browser constantly uses https so until I discovered that and selected Darksky api, it was all smooth sailing.
Using Skycon, didn’t realised I had to create a new object did that and it is plain sailing again.
oh and the fact that I had to put “https://crossorigin.me/” in front of the API url.

I am however getting a lag displaying the weather from Darksky API, wonder if anyone knows the reason?

Here is my code.

Thanks,
Jasmine

Project Link - https://codepen.io/bdimic/full/MoEbrJ/

Here is my project link. https://codepen.io/joshbeckerleg/full/WOEjwv/

Any feedback is welcomed!

Josh

3 Likes

looks much better in browser fyi

Ok here is the cleaned up version! Problem is I still cant get the background image to change!
Help, any suggestions?

https://codepen.io/DaBubbla/full/QgarRB/

Project Link - https://codepen.io/smasher/full/MpLKXM/
Made simple app
I am new so,
Kindly review [like code syntax,readability,more content,responsiveness] I will pay attention to every review.
Thanks

So I’m wondering about this code snippet here: geolocation.getCurrentPosition(function(position) {
latt = position.coords.latitude;
long = position.coords.longitude;

Does geolocatio still work in Chrome, because I thought it was depreciated? As you can see, I can’t see my location with this app.

Really love your animations and layout! The moving sun and changing clouds are awesome!

1 Like