Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fire 1 - Weather app #16

Open
wants to merge 81 commits into
base: main
Choose a base branch
from
Open

Fire 1 - Weather app #16

wants to merge 81 commits into from

Conversation

simfrisk
Copy link

Here is our weather app! Hope you like it!

/Tilde, Ida, Jonny and Simon

simfrisk and others added 30 commits March 17, 2025 19:46
…d made som functions for fromat time and capital first letter
"basic style created added  Sun to style IMG"
Added a function to fetch the forecast data
…L, updated error handling when user search on a city that not match the api data
added function to get the users location
added timezon function to offset the time. I have not still gotten th…
@simfrisk
Copy link
Author

We will try to add a loading message before we have gotten the information from the API

@JennieDalgren
Copy link

@simfrisk @KidFromCalifornia @tildetilde @Idahel Can you add your Netlify-link ?

@simfrisk
Copy link
Author

@JennieDalgren

https://weather-fire.netlify.app/

Copy link

@JennieDalgren JennieDalgren left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants