Skip to content

My GitHub tracker#22

Open
madeleinesvensson wants to merge 14 commits intoTechnigo:mainfrom
madeleinesvensson:main
Open

My GitHub tracker#22
madeleinesvensson wants to merge 14 commits intoTechnigo:mainfrom
madeleinesvensson:main

Conversation

@madeleinesvensson
Copy link
Copy Markdown

No description provided.

Copy link
Copy Markdown

@Ingetora Ingetora left a comment

Choose a reason for hiding this comment

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

Over all, a very nice project! You have completed all of the requirements for the blue level:

  • All of your Technico repos are represented.
  • You show your user name and profile pic.
  • You have a push for each latest update.
  • You present the names of the default branches.
  • URL to actual repo.
  • I can see the number of commits.
  • Your project is responsive

Looking at your code, I think you have achieved a lot from only few lines of code! Your code is clean written and logic. I think you have done a great job on the responsiveness, and yours is one of the more stylish ones I've seen this week. I would prefer if the links to the repos opened up in new tabs when you click on them, but that is a matter of personal preference. Keep up the good work!

Comment thread code/script.js Outdated
//const PULLREQUEST_URL = `https://api.github.com/repos/technigo/${repo.name}/pulls`;
const container = document.getElementById("projects");
const userInformation = document.getElementById("user-information");
const commitsContainer = document.getElementById("commit-container");
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 naming of the variables, it's evident what you are doing here.

Comment thread code/script.js
weekday: "short",
year: "numeric",
month: "short",
day: "numeric",
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I really like how you display date and time with your projects!

Comment thread code/style.css
flex-wrap: nowrap;
align-items: center;
justify-content: center;
}
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Nice example of stylish but simple design.

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