Skip to content
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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

feat: get gitlog by command #149

wants to merge 1 commit into from

Conversation

0x-ck
Copy link

@0x-ck 0x-ck commented Jan 27, 2025

I use git cmd to fetch git commit history instead of github api

Copy link

netlify bot commented Jan 27, 2025

Deploy Preview for tansu canceled.

Name Link
🔨 Latest commit 8c61771
🔍 Latest deploy log https://app.netlify.com/sites/tansu/deploys/679778c1b2c51300087b03bf

Copy link

netlify bot commented Jan 27, 2025

Deploy Preview for testnet-tansu ready!

Name Link
🔨 Latest commit 8c61771
🔍 Latest deploy log https://app.netlify.com/sites/testnet-tansu/deploys/679778c13a2cb40008400916
😎 Deploy Preview https://deploy-preview-149--testnet-tansu.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 79
Accessibility: 88
Best Practices: 92
SEO: 85
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Owner

@tupui tupui left a 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)
Copy link
Owner

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";
Copy link
Owner

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(
Copy link
Owner

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}`)
Copy link
Owner

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.

@tupui tupui linked an issue Jan 28, 2025 that may be closed by this pull request
@tupui tupui added enhancement New feature or request javascript Everything JS dapp dApp and UI side of things labels Jan 28, 2025
export async function GET({ url }) {
console.log("url", url);
const username = "tupui";
const repo = "soroban-versioning";
Copy link
Contributor

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);
Copy link
Contributor

@0xExp-po 0xExp-po Jan 27, 2025

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dapp dApp and UI side of things enhancement New feature or request javascript Everything JS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

History without GitHub
3 participants