-
Notifications
You must be signed in to change notification settings - Fork 131
Project-github-tracker #118
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
Katarina821
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi Jens! Great job this week and good work solving the commits issue! Your code is clean and easy to follow, nice structured with the different functions for the commits. Nicely done:)!
| picture = data.avatar_url | ||
| let profilePicture = `<div class = "profile"><h1>${username}</h1> <img class = "photo" src="${picture}" /> | ||
| </div>`; | ||
| return (projects.innerHTML = profilePicture) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Interesting solution with the variable and the the returnstatment, tought me something new!
| </div> | ||
| ` | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe here you could look over your structure a bit and remove blanks space so that it follows codeguidlines
| fetch(commits) | ||
| .then(response => response.json()) | ||
| .then(data =>{ | ||
| document.getElementById(`${myRepoName}`).innerHTML += data.length |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You solved it, good job:)!!
No description provided.