Welcome to the Treehouse Community

The Treehouse Community is a meeting place for developers, designers, and programmers of all backgrounds and skill levels to get support. Collaborate here on code errors or bugs that you need feedback on, or asking for an extra set of eyes on your latest project. Join thousands of Treehouse students and alumni in the community today. (Note: Only Treehouse students can comment or ask questions, but non-students are welcome to browse our conversations.)

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and a supportive community. Start your free trial today.

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,717 Points
Marcus Parsons
Marcus Parsons
15,717 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,717 Points

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