-
Notifications
You must be signed in to change notification settings - Fork 5
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
feat: get gitlog by command #149
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for tansu canceled.
|
✅ Deploy Preview for testnet-tansu ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
Thank you for working on this 😃 A few things to start with the iterative process.
setLoading(true); | ||
setError(""); | ||
console.log("author", projectRepoInfo.author) | ||
console.log("repo", projectRepoInfo.repository) |
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.
Clean all logs
|
||
|
||
|
||
// import React from "react"; |
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.
Why is all that commented?
commits: commits as FormattedCommit[], | ||
})); | ||
} | ||
// async function getCommitHistory( |
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.
To be deleted an not commented if not used anymore
setError(""); | ||
console.log("author", projectRepoInfo.author) | ||
console.log("repo", projectRepoInfo.repository) | ||
const response = await axios.get(`/api/commits?username=${projectRepoInfo.author}&repo=${projectRepoInfo.repository}`) |
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.
To be clear, I don't want to have a backend API. Fetching the repo has to happen client side and be cached there.
export async function GET({ url }) { | ||
console.log("url", url); | ||
const username = "tupui"; | ||
const repo = "soroban-versioning"; |
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 values is hard coded here
// const repo = params.get('repo') | ||
|
||
console.log("username", username); | ||
console.log("repo", repo); |
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.
Please remove unnecessary console log
I use git cmd to fetch git commit history instead of github api