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 Asynchronous Programming with JavaScript Exploring Async/Await Convert Promise Handling to Async/Await

Event listener button won't go away after clicking it

So after following the lesson, I tested the code to see that it works but the button doesn't disappear. Any thoughts?

const astrosUrl = 'http://api.open-notify.org/astros.json';
const wikiUrl = 'https://en.wikipedia.org/api/rest_v1/page/summary/';
const peopleList = document.getElementById('people');
const btn = document.querySelector('button');

// Handle all fetch requests
async function getPeopleInSpace(url) {
  const peopleResponse = await fetch(url);
  const peopleJSON = await peopleResponse.json();

  const profiles = await peopleJSON.people.map( async (person) => {
    const craft = person.craft;
    const profileResponse = await fetch(wikiUrl + person.name);
    const profileJSON = await profileResponse.json();

  return { ...profileJSON, craft};
  });
  return Promise.all(profiles);
}

// Generate the markup for each profile
function generateHTML(data) {
  data.map( person => {
    const section = document.createElement('section');
    peopleList.appendChild(section);
    section.innerHTML = `
      <img src=${person.thumbnail.source}>
      <span>${person.craft}</span>
      <h2>${person.title}</h2>
      <p>${person.description}</p>
      <p>${person.extract}</p>
    `;
  });
}

btn.addEventListener('click', async (event) => {
  event.target.textContent = "Loading...";

  const astros = await getPeopleInSpace(astrosUrl);
  generateHTML(astros);
  event.target.remove();
});

1 Answer

Check the console. You have an error here:

<img src=${person.thumbnail.source}>

This is due to ambiguity with the names of two astronauts not retrieving the correct page: Alexander Skvortsov and Andrew Morgan.

I modified getPeopleInSpace() to use the disambiguated pages by using Aleksandr_Skvortsov_(cosmonaut) and Andrew_R._Morgan respectively.

// Handle all fetch requests
async function getPeopleInSpace(url) {
  const peopleResponse = await fetch(url);
  const peopleJSON = await peopleResponse.json();

  const profiles = peopleJSON.people.map( async (person) => {
    const craft = person.craft;

    var fetchurl = wikiUrl + person.name

    if (person.name == "Alexander Skvortsov") {
       fetchurl = "https://en.wikipedia.org/api/rest_v1/page/summary/Aleksandr_Skvortsov_(cosmonaut)"
    }   

    if (person.name == "Andrew Morgan") {
       fetchurl = "https://en.wikipedia.org/api/rest_v1/page/summary/Andrew_R._Morgan"
    }

    const profileResponse = await fetch(fetchurl);
    const profileJSON = await profileResponse.json();

    return { ...profileJSON, craft };
  });

  return Promise.all(profiles);
}
Seokhyun Wie
seal-mask
.a{fill-rule:evenodd;}techdegree seal-36
Seokhyun Wie
Full Stack JavaScript Techdegree Graduate 21,594 Points

KRIS NIKOLAISEN Hi Kris, thanks for the answer. Just to understand it more clearly, can you explain why it does not disappear in a structural way? What I mean is that before we put async and await, the event.target(=button).remove() worked no matter what error occurs in the middle. Thanks in advance.