Skip to content
This repository has been archived by the owner on Oct 26, 2020. It is now read-only.

Leidas week 8 homework #1005

Open
wants to merge 1 commit into
base: manchester3
Choose a base branch
from

Conversation

Leidagandy
Copy link

Your Details

Your Name: Leida Goncalves
Your City: Manchester
Your Slack Name: Leida Gandy

Homework Details

Module: JS Week III week 8
Week: 8

.catch((error) => {
console.error(error);
});

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well done Leida.

console.log("error!!!");
console.error(error);
});
}

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good Job Leida, you forgot to implement error handling code at the fetched data. Implement that.

console.error(error);
});
}
getJoke();

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This function is correct, calling the function at line 20 is not appropriate. what you could do separate the fetching and the getJoke then call getJoke in the .then() or the easiest way apply the innerText in the .then() . Work on that.

@Abdoulrazack95 Abdoulrazack95 added the reviewed A mentor has reviewed this PR label Jul 2, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
reviewed A mentor has reviewed this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants