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

DOM manipulation #49

Closed
LaLeonie opened this issue Nov 29, 2019 · 1 comment
Closed

DOM manipulation #49

LaLeonie opened this issue Nov 29, 2019 · 1 comment

Comments

@LaLeonie
Copy link

When I use the Heroku link, the data from the API call isn't appended to the site but instead, I am redirected to another tab that holds the JSON object.

But seeing your DOM.js file, it looks like you're on the right track to implement the functionality that the data is added to the DOM.

Just remember that you'll need to clear the weather data in the DOM before each new API call, otherwise you just end up with a list of data from all previous searches.

@redahaq redahaq closed this as completed Nov 29, 2019
@redahaq
Copy link
Collaborator

redahaq commented Nov 29, 2019

Thank you! Yep we were just missing a '.weather' at the end!

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

No branches or pull requests

2 participants