JavaScript AJAX Basics (retiring) AJAX and APIs Call the jQuery $.getJSON method

Albert Collazo-Delvalle
Albert Collazo-Delvalle
7,223 Points

can anyone spot the issue with my $.getJSON() method on line 8?

Bummer: You didn't correctly call the $.getJSON() method. To use it, pass it three arguments: the URL, the data you want to send to the server, and the callback function.

weather.js
$(document).ready(function() {

  var weatherAPI = "http://api.openweathermap.org/data/2.5/weather";
  var data = {
    q : "Portland,OR",
    units : "metric"
  };
  $.getJSON(weatherAPI, data, function (weatherReport) {
    $('#temperature').text(weatherReport.main.temp);
  });
});
index.html
<!DOCTYPE html>
<html>
<head>
  <meta charset="utf-8">
  <title>What's the Weather Like?</title>
  <script src="jquery.js"></script>
  <script src="weather.js"></script>
</head>
<body>
  <div id="main">
    <h1>Current temperature: <span id="temperature"></span>&deg;</h1>
  </div>
</body>
</html>

2 Answers

Clayton Perszyk
MOD
Clayton Perszyk
Treehouse Moderator 46,143 Points

You're code is correct, but to complete the challenge pass the function that is already present to getJSON (i.e. do not use an anonymous function).

Albert Collazo-Delvalle
Albert Collazo-Delvalle
7,223 Points

hey Clayton, thanks for the response. Unfortunately I have tried that approach as well but still got an error message.

Clayton Perszyk
MOD
Clayton Perszyk
Treehouse Moderator 46,143 Points

strange this passes for me

$(document).ready(function() {

  var weatherAPI = 'http://api.openweathermap.org/data/2.5/weather';
  var data = {
    q : "Portland,OR",
    units : "metric"
  };
  function showWeather(weatherReport) {
    $('#temperature').text(weatherReport.main.temp);
  }

  $.getJSON(weatherAPI, data, showWeather);

});
Albert Collazo-Delvalle
Albert Collazo-Delvalle
7,223 Points

i called the call back function wrong! i had: $.getJSON(weatherAPI, data, showWeather()); Thanks a lot Clayton, much appreciated!