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

week8 H.W is done #1013

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

Conversation

Fakher-syr
Copy link

Your Details

Your Name:
Your City:
Your Slack Name:

Homework Details

Module:
Week:

Copy link

@Ashaghel Ashaghel left a comment

Choose a reason for hiding this comment

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

Great work

})
.then(function (greeting) {
document.getElementById("greeting-text").textContent = greeting;

Choose a reason for hiding this comment

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

perfect !

fetch("https://dog.ceo/api/breeds/image/random")
.then((response) => response.json())
.then((data) => {
dogImage1.innerHTML = `<img src="${data.message}"/>`;

Choose a reason for hiding this comment

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

excellent !

@@ -0,0 +1,8 @@
fetch(`https://xkcd.now.sh/?comic=latest`)
.then((response) => response.json())
.then((response) => {

Choose a reason for hiding this comment

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

you need to add some error handling

@Ashaghel Ashaghel added the reviewed A mentor has reviewed this PR label Jul 10, 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.

3 participants