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) jQuery and AJAX The Office Status Project Revisited

kabir k
PLUS
kabir k
Courses Plus Student 18,036 Points
Posted by kabir k
kabir k
Courses Plus Student 18,036 Points

My employee list is not showing up in the preview

I followed along with the video but for some reason my employee list is not showing up in the preview.

Can someone take a look at it and let me know what's wrong?

$(document).ready(function () { 
  var url = '../data/employees.json';
  $.getJSON(url, function (response) {
    var statusHTML = '<ul class="bulleted">';
    $.each(response, function (index, employee) { 
      if (employee.inoffice === true) {
        statusHTML += <li class="in">;
      } else {
        statusHTML += <li class="out">;
      } 
      statusHTML += employee.name + '</li>';
    });
    statusHTML += '</ul>'; 
    $('#employeeList').html(statusHTML); 
  }); // end getJSON
}); // end ready

2 Answers

kabir k
PLUS
kabir k
Courses Plus Student 18,036 Points
kabir k
kabir k
Courses Plus Student 18,036 Points

Never mind. I found the problem. I forgot to put the <li> tags in the if/else statements in quotes.

Marcus Parsons
Marcus Parsons
15,718 Points
Marcus Parsons
Marcus Parsons
15,718 Points

Hey Kabir,

It looks like you're missing quotes around the statusHTML inside of the each function. Wrap those in single quotes and you should be good to go.

$(document).ready(function () { 
  var url = '../data/employees.json';
  $.getJSON(url, function (response) {
    var statusHTML = '<ul class="bulleted">';
    $.each(response, function (index, employee) { 
      if (employee.inoffice === true) {
        //added single quotes
        statusHTML += '<li class="in">';
      } else {
        //added single quotes
        statusHTML += '<li class="out">';
      } 
      statusHTML += employee.name + '</li>';
    });
    statusHTML += '</ul>'; 
    $('#employeeList').html(statusHTML); 
  }); // end getJSON
}); // end ready
kabir k
kabir k
Courses Plus Student 18,036 Points
kabir k
kabir k
Courses Plus Student 18,036 Points

Thanks, Marcus. Yeah, I realized that as well. Thanks, anyway.

Marcus Parsons
Marcus Parsons
15,718 Points

Oh haha I see your comment now. Glad you fixed it, though!