Skip to content

Lisas github tracker#90

Open
lisabergstrom wants to merge 11 commits intoTechnigo:mainfrom
lisabergstrom:main
Open

Lisas github tracker#90
lisabergstrom wants to merge 11 commits intoTechnigo:mainfrom
lisabergstrom:main

Conversation

@lisabergstrom
Copy link
Copy Markdown

No description provided.

Copy link
Copy Markdown

@joannaringqvist joannaringqvist left a comment

Choose a reason for hiding this comment

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

Great job with the GitHub tracker! Think your code was very clean and easy to follow and also good that you tried the token thing! Super!
🥳 🥳 🥳

const API_REPOS = `https://api.github.com/users/${username}/repos`;
const API_PROFILE = `https://api.github.com/users/${username}`;

// T O K E N
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Great that you fixed this with the token!

const userProfile = document.getElementById("userProfile");
const projects = document.getElementById("projects");

// P R O F I L E
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Also good that you had the profile in an own function. Makes it more clean.

// Indicates a pull request or not.
// If yes - call fetchCommits function
// If no - inform user that no pull request was yet done
if (myPullRequest) {
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This was good

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