Welcome to the Treehouse Community

Want to collaborate on code errors? Have bugs you need feedback on? Looking for an extra set of eyes on your latest project? Get support with fellow developers, designers, and programmers of all backgrounds and skill levels here with the Treehouse Community!

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and join thousands of Treehouse students and alumni in the community today.

Start your free trial

JavaScript AJAX Basics (retiring) AJAX and APIs Displaying the Photos

Not understanding why my code isn't working. Using a Mac. Photos not loading to the page.

$(document).ready(function () { $('button').click(function () { $("button").removeClass("selected"); $(this).addClass("selected"); var flickerAPI = "http://api.flickr.com/services/feeds/photos_public.gne?jsoncallback=?"; var animal = $(this).text(); var flickrOptions = { tags: animal, format: "json" };
function displayPhotos(data) { var photoHTML = '<ul>'; $.each( data.items, function (i, photo) { photoHTML += '<li class="grid-25 tablet-grid-50">'; photoHTML += '<a href="' + photo.link + '" class="image">'; photoHTML += '<img src="' + photo.media.m + '"></a></li>'; }); photoHTML = '</ul>'; $('#photos').html(photoHTML); } $.getJSON(flickerAPI flickrOptions, displayPhotos); });
});// end ready

Regena Koeshall
.a{fill-rule:evenodd;}techdegree seal-36
Regena Koeshall
Front End Web Development Techdegree Graduate 21,143 Points

The only two things I see with your code is that you are trying to link to the Flickr API using http instead of https and you are missing the first comma in $.getJSON(flickerAPI, flickrOptions, displayPhotos);

Hope that helps.

Ryne Ziemba
Ryne Ziemba
9,967 Points

Put animal in quotes.

1 Answer

Pedro Dasilva
Pedro Dasilva
8,291 Points

Hey Dean I think that your main mistake here is not closing the <ul> , use += operator to add correctly </ul> closing tag to var photoHTML, right now it seems that you make photoHTML = </ul> only