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

incomplete homework week 3 #37

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

andresjv
Copy link
Contributor

No description provided.

$.get(weatherUrl+apiKey, function(r) {
var s = JSON.stringify(r);
console.log(s);
});
Copy link
Owner

Choose a reason for hiding this comment

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

Now that you have data to work with, open console in Chrome and see what's logged to the console. In order to play with that result, parse the json into an object. JSON data is sent and received as a string, but has to be parsed into objects before it can be used. Once you do that, see if you can traverse that object in console to find the temperature.



$( "#create" ).click(function() {
alert( "Handler for .click() called." );
Copy link
Owner

Choose a reason for hiding this comment

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

This is a great way to debug 👍

"name" : "cat",
"species" : "Felis catus",
"breed" : "Tonkinese",
"noise" : "meow"
Copy link
Owner

Choose a reason for hiding this comment

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

This is... an oddly clean looking object... except it's in my folder and not yours. :P
Unless you want to give me all your cats, dogs, and monkeys, go ahead and move anything you accidentally dropped into my folder back into yours, then git add, commit, and push back up. Your pull request will automatically update.

@Borgaard
Copy link
Owner

Some assignments are still in my directory. :\

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.

None yet

2 participants