Skip to content

Jessica Babcock HW - Weather API #9

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

jbabs22
Copy link

@jbabs22 jbabs22 commented Apr 17, 2023

Comprehension

  1. I actually felt this deliverable seemed a lot easier than it was but not in the actual JS code, but more so in the displaying of that code to the UI. I sucessfully gave the user the ability to enable/disable air quality info and I didn't like the way I was displaying the different numbers, and therefore consolidated down to one value of the air quality to bring in. I am finding out more that if I do not setup my HTML data in a more efficient manner, I will struggle a bit more with calling things in JS or trying to structure things in CSS, so practicing more and more is really making things come together, and sometimes later than I'd like.

Completion

  1. I have met all requirements + any bonuses for this deliverable (just later than the deadline 👎 )

Additional Comments

I, myself am looking to give myself more practice with the supporting files to better work efficiently with my JS code. In addition, I believe practicing and trying different things is currently working for me, but there are times, I am almost asking myself why does this work, and YAY at least there ... yet. Loving every second of it though!

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.

2 participants