-
Notifications
You must be signed in to change notification settings - Fork 130
Aschwin Siaila #37
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?
Aschwin Siaila #37
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,13 +1,11 @@ | ||
| # GitHub Tracker | ||
|
|
||
| Replace this readme with your own information about your project. | ||
|
|
||
| Start by briefly describing the assignment in a sentence or two. Keep it short and to the point. | ||
| This week was quite a challenge because I was traveling. Got a lot of help from the recorded videos. So thank God for this week! | ||
|
|
||
| ## The problem | ||
|
|
||
| Describe how you approached to problem, and what tools and techniques you used to solve it. How did you plan? What technologies did you use? If you had more time, what would be next? | ||
| to understand which and what data comes from where or what. Using console.log(data) explained alot. | ||
|
|
||
| ## View it live | ||
|
|
||
| Every project should be deployed somewhere. Be sure to include the link to the deployed project so that the viewer can click around and see what it's all about. | ||
| https://practical-fermi-1dc0a6.netlify.app/ |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,4 +1,24 @@ | ||
| //DOM-selector for the canvas 👇 | ||
| const ctx = document.getElementById('chart').getContext('2d') | ||
| const ctx = document.getElementById("chart").getContext("2d"); | ||
|
|
||
| //"Draw" the chart here 👇 | ||
| const drawProgressChart = (repos) => { | ||
| const completedProjects = repos.length; | ||
| const totalProjects = 20; | ||
|
|
||
| const labels = repos.map((repo) => repo.name); | ||
| const data = repos.map((repo) => repo.size); | ||
|
|
||
| const chart = new Chart(ctx, { | ||
| type: "doughnut", | ||
| data: { | ||
| labels: ["Completed Projects", "Projects left to build"], | ||
| datasets: [ | ||
| { | ||
| data: [completedProjects, totalProjects - completedProjects], | ||
| backgroundColor: ["blue", "red"], | ||
| hoverOffset: 4, | ||
| }, | ||
| ], | ||
| }, | ||
| }); | ||
| }; |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,21 +1,31 @@ | ||
| <!DOCTYPE html> | ||
| <html lang="en"> | ||
| <head> | ||
| <meta charset="UTF-8"> | ||
| <meta http-equiv="X-UA-Compatible" content="IE=edge"> | ||
| <meta name="viewport" content="width=device-width, initial-scale=1.0"> | ||
| <title>Project GitHub Tracker</title> | ||
| <link rel="stylesheet" href="./style.css" /> | ||
| </head> | ||
| <body> | ||
| <h1>GitHub Tracker</h1> | ||
| <h2>Projects:</h2> | ||
| <main id="projects"></main> | ||
| <head> | ||
| <meta charset="UTF-8" /> | ||
| <meta http-equiv="X-UA-Compatible" content="IE=edge" /> | ||
| <meta name="viewport" content="width=device-width, initial-scale=1.0" /> | ||
| <link rel="preconnect" href="https://fonts.googleapis.com" /> | ||
| <link rel="preconnect" href="https://fonts.gstatic.com" crossorigin /> | ||
| <link | ||
| href="https://fonts.googleapis.com/css2?family=IBM+Plex+Sans:ital,wght@0,100;0,300;0,400;0,700;1,100;1,400&display=swap" | ||
| rel="stylesheet" | ||
| /> | ||
| <link rel="stylesheet" href="./style.css" /> | ||
| <title>Project GitHub Tracker</title> | ||
| <script src="https://cdn.jsdelivr.net/npm/chart.js"></script> | ||
| </head> | ||
| <body> | ||
| <h1>GitHub Tracker Aschwin Siaila</h1> | ||
| <h2>Name and Picture:</h2> | ||
| <main id="nameandpicture"></main> | ||
| </div> | ||
| <h2>Projects:</h2> | ||
| <main id="projects"></main> | ||
|
|
||
| <!-- This will be used to draw the chart 👇 --> | ||
| <canvas id="chart"></canvas> | ||
|
|
||
| <script src="./script.js"></script> | ||
| <script src="./chart.js"></script> | ||
| </body> | ||
| </html> | ||
| <!-- This will be used to draw the chart 👇 --> | ||
| <h2>Diagram:</h2> | ||
| <canvas id="chart"></canvas> | ||
| <script src="./script.js"></script> | ||
| <script src="./chart.js"></script> | ||
| </body> | ||
| </html> |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,68 @@ | ||
| const USER = "AschwinSiaila"; | ||
| const REPOS_URL = `https://api.github.com/users/${USER}/repos`; | ||
| const nameAndPicture = document.getElementById("nameandpicture"); | ||
| const projectsContainer = document.getElementById("projects"); | ||
|
|
||
| const NameandPicture = () => { | ||
| const personal_ID = `https://api.github.com/users/${USER}`; | ||
| fetch(personal_ID) | ||
| .then((res) => res.json()) | ||
| .then((data) => { | ||
| global__UserData = data; | ||
| }) | ||
| .then(() => NameAndPicture()); | ||
| }; | ||
|
|
||
| const NameAndPicture = () => { | ||
| const picture = global__UserData.avatar_url; | ||
|
|
||
| nameAndPicture.innerHTML += ` | ||
| <img class="profile-pic" src="${picture}" alt="Picture of gitHub-user"/> | ||
| <div class="profile-name">${USER}</div> | ||
| `; | ||
| }; | ||
|
|
||
| const fetchRepositories = () => { | ||
| fetch(REPOS_URL) | ||
| .then((res) => res.json()) | ||
| .then((data) => { | ||
| const technigoRepositories = data.filter((repo) => repo.name.includes("project-") && repo.fork); | ||
|
|
||
| console.log(technigoRepositories); | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It's great that you use console logs! They're really helpful! Don't forget to delete them after you finished with the project :) |
||
|
|
||
| technigoRepositories.forEach((repo) => { | ||
| projectsContainer.innerHTML += ` | ||
| <div class="project-info"> | ||
| <a class="project-links" href="${repo.html_url}">${repo.name} with default branch ${repo.default_branch}</a> | ||
| <p class="push">Recent push: ${new Date(repo.pushed_at).toDateString()}</p> | ||
| <a href="${repo.html_url}"><p class="commits" id="commit-${repo.name}">Commits amount: </p></a> | ||
|
|
||
| </div> | ||
| `; | ||
| }); | ||
|
|
||
| fetchPullRequestArray(technigoRepositories); | ||
| drawProgressChart(technigoRepositories); | ||
|
Comment on lines
+44
to
+45
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Great names for the functions! 💯 |
||
| }); | ||
| }; | ||
|
|
||
| const fetchPullRequestArray = (allrepositories) => { | ||
| allrepositories.forEach((repo) => { | ||
| fetch(`https://api.github.com/repos/Technigo/${repo.name}/pulls`) | ||
| .then((res) => res.json()) | ||
| .then((data) => { | ||
| const myPullRequest = data.find((pull) => pull.user.login === repo.owner.login); | ||
| fetchCommits(myPullRequest.commits_url, repo.name); | ||
| }); | ||
| }); | ||
| }; | ||
| const fetchCommits = (myCommitsUrl, myRepoName) => { | ||
| fetch(myCommitsUrl) | ||
| .then((res) => res.json()) | ||
| .then((data) => { | ||
| document.getElementById(`commit-${myRepoName}`).innerHTML += data.length; | ||
| }); | ||
| }; | ||
|
|
||
| NameandPicture(); | ||
| fetchRepositories(); | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,3 +1,101 @@ | ||
| body { | ||
| background: #FFECE9; | ||
| } | ||
| background: #d6c5c2; | ||
| } | ||
|
|
||
| #nameandpicture { | ||
| position: relative; | ||
| max-width: 100%; | ||
| height: auto; | ||
| } | ||
|
|
||
| #projects { | ||
| display: grid; | ||
| grid-template-columns: auto; | ||
| gap: 10px; | ||
| } | ||
|
|
||
| .profile-pic { | ||
| border-radius: 30px; | ||
| max-width: 100%; | ||
| height: auto; | ||
| } | ||
|
|
||
| .profile-name { | ||
| font-size: 100%; | ||
| font-family: Cambria, Cochin, Georgia, Times, "Times New Roman", serif; | ||
| background-color: rgb(56, 172, 88); | ||
| color: white; | ||
| padding: 20px; | ||
| max-width: 100%; | ||
| height: auto; | ||
| } | ||
|
|
||
| .project-info { | ||
| font-family: Verdana, Geneva, Tahoma, sans-serif; | ||
| color: white; | ||
| max-width: 100%; | ||
| height: auto; | ||
| border: 2px solid black; | ||
| padding: 10px; | ||
| background-color: rgb(130, 110, 109); | ||
| } | ||
|
|
||
| .project-links { | ||
| color: white; | ||
| font-size: 12px; | ||
| max-width: 100%; | ||
| height: auto; | ||
| } | ||
|
|
||
| .push { | ||
| background-color: teal; | ||
| padding: 20px; | ||
| max-width: 100%; | ||
| height: auto; | ||
| border: 2px solid black; | ||
| } | ||
|
|
||
| .commits { | ||
| background-color: turquoise; | ||
| padding: 20px; | ||
| max-width: 100%; | ||
| height: auto; | ||
| border: 2px solid black; | ||
| } | ||
|
|
||
| @media (min-width: 768px) { | ||
| .profile-name { | ||
| font-family: Cambria, Cochin, Georgia, Times, "Times New Roman", serif; | ||
| background-color: rgb(56, 172, 88); | ||
| color: white; | ||
| position: absolute; | ||
| top: 8px; | ||
| left: 16px; | ||
| font-size: 20px; | ||
| } | ||
|
|
||
| #projects { | ||
| display: grid; | ||
| grid-template-columns: auto auto; | ||
| gap: 10px; | ||
| padding: 10px; | ||
| } | ||
| } | ||
| @media (min-width: 992px) { | ||
| .profile-name { | ||
| font-family: Cambria, Cochin, Georgia, Times, "Times New Roman", serif; | ||
| background-color: rgb(56, 172, 88); | ||
| color: white; | ||
| position: absolute; | ||
| top: 8px; | ||
| left: 16px; | ||
| font-size: 20px; | ||
| } | ||
|
|
||
| #projects { | ||
| display: grid; | ||
| grid-template-columns: auto auto auto auto; | ||
| gap: 10px; | ||
| padding: 10px; | ||
| } | ||
| } |
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.
These functions have the same name. In the first one you fetching the data and another one you creating HTML. If you combine those two you can have only one function and have mu less lines of code but the same functionality :) just copy everything that you have in your second function and put it in your first one after line 13