-
Notifications
You must be signed in to change notification settings - Fork 12
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
Connect to Weather API #28
base: main
Are you sure you want to change the base?
Changes from 1 commit
08fa292
dac08f8
87d8e92
a18dd57
be36012
85d523f
ebb724f
accfe59
544ee29
2cbd25e
8b2d947
fa398df
dc4e844
f47a206
925f7de
7c3b35c
050e635
4a6f098
b458d4b
42f58a9
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,53 @@ | ||
import React from "react"; | ||
import storm from "../../img/weather-icons/storm.svg"; | ||
import drizzle from "../../img/weather-icons/drizzle.svg"; | ||
import rain from "../../img/weather-icons/rain.svg"; | ||
import snow from "../../img/weather-icons/snow.svg"; | ||
import fog from "../../img/weather-icons/fog.svg"; | ||
import clear from "../../img/weather-icons/clear.svg"; | ||
import partlyCloudy from "../../img/weather-icons/partlycloudy.svg"; | ||
import mostlyCloudy from "../../img/weather-icons/mostlycloudy.svg"; | ||
|
||
const images = { | ||
storm: { src: storm, alt: "storm" }, | ||
drizzle: { src: drizzle, alt: "drizzle" }, | ||
rain: { src: rain, alt: "rain" }, | ||
snow: { src: snow, alt: "snow" }, | ||
fog: { src: fog, alt: "fog" }, | ||
clear: { src: clear, alt: "clear" }, | ||
partlyCloudy: { src: partlyCloudy, alt: "partly cloudy" }, | ||
mostlyCloudy: { src: mostlyCloudy, alt: "mostly cloudy" }, | ||
}; | ||
|
||
|
||
function selectImage(weatherId){ | ||
if (weatherId < 300) { | ||
return images.storm; | ||
} else if (weatherId < 499) { | ||
return images.drizzle; | ||
} else if (weatherId < 599) { | ||
return images.rain; | ||
} else if (weatherId < 699) { | ||
return images.snow; | ||
} else if (weatherId < 799) { | ||
return images.fog; | ||
} else if (weatherId === 800) { | ||
return images.clear; | ||
} else if (weatherId === 801) { | ||
return images.partlyCloudy; | ||
} | ||
return images.mostlyCloudy; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Nice code here @zkhing 🙌🏼
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Please make this small change, then I will approve this PR, and we can merge it in. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Thank you. Actually, my tech buddy kindly helped me with that part. :) And I added unknown.svg image now. |
||
} | ||
|
||
const WeatherIcon = ({weatherId}) =>{ | ||
|
||
const image = selectImage(weatherId) | ||
|
||
return ( | ||
<div className="weather-img"> | ||
<img src={image.src} alt={image.alt} /> | ||
</div> | ||
); | ||
} | ||
|
||
export default WeatherIcon; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can delete this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi Lucy, sorry for late reply! yes, I have deleted it.