Skip to content

Conversation

@rijad90
Copy link

@rijad90 rijad90 commented Feb 27, 2022

No description provided.

Copy link

@TereseClaesson TereseClaesson left a comment

Choose a reason for hiding this comment

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

A clear, well structured and consistent kod, with good comments. Easy to follow your code and it looks great. Well done!

.then((data) => {
const technigoRepo = data.filter(
// FILTERS MY FORKED REPOS ONLY TO DESPLAY THE ONES NAMNED "project-"
(repo) => repo.name.includes('project-') && repo.fork

Choose a reason for hiding this comment

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

You have done a great job and I see that you have used includes(),I will learn from that ;) It's also possible to use startsWith() when searching for 'projects-', that is what i did in my project.


//DOM SELECTOR
const projects = document.getElementById('projects-container')
//FETCH API AND FILTERS REPO. INJECTS REPO URL, NAME, DEF BRANCH, PUSHED DATE AND COMMIT

Choose a reason for hiding this comment

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

clear and concise comment!

const GITHUB_USER_API = `https://api.github.com/users/${username}`
const GITHUB_REPOS_API = `https://api.github.com/users/${username}/repos`

//FETCH API TO PROFILE. INJECTS PIC,USERNAME,BIO

Choose a reason for hiding this comment

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

clear and concise comment!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants