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

Added stars/forks feature #194

Merged
merged 3 commits into from
Oct 23, 2022
Merged

Conversation

harsh0620
Copy link
Contributor

PR Title

The purpose of this Pull Request is to fix #189

Description

  • Stars and Forks of the repo will be shown below the navbar

How you solved

  • Use github's Octokit/rest package to get repo details
  • and swr package to update the details without refreshing
  • One has to assign access token in .env as GITHUB_AUTH_TOKEN=
  • For star and fork icon use react-icon package

Screenshots

image

Checklist

  • I have Made this contribution as per the CONTRIBUTING guide in this repo
  • I have tested in local Environment.
  • I have made the fix as per issue converstaion
  • I have starred the repository ⭐

@vercel
Copy link

vercel bot commented Oct 22, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
tailwindcss-ui-components ✅ Ready (Inspect) Visit Preview Oct 23, 2022 at 6:25AM (UTC)

@jsvigneshkanna
Copy link
Owner

@harsh0620 , whose auth token is there in env file. Should i need to update that to mine?

@harsh0620
Copy link
Contributor Author

@jsvigneshkanna , yes your auth token should be used here.

@jsvigneshkanna
Copy link
Owner

@harsh0620 , can you please check the conflicts ( i am not sure because those are in yarn and packages)

Also check for other open issues too

@harsh0620
Copy link
Contributor Author

@jsvigneshkanna I try to resolve the conflicts by maintaining same code but its not resolving conflicts.
I think It will not create any problem as the change in package and yarn file is emerging as I installed two packages.
Please merge it as webapp is up and running fine.

@jsvigneshkanna
Copy link
Owner

@harsh0620 sure will check merge in sometime

Please check for other open issues

@jsvigneshkanna
Copy link
Owner

@harsh0620 ,
Can you raise new PR again with same changes ?

@harsh0620
Copy link
Contributor Author

@jsvigneshkanna , I can but I only know to do this by writing same code after cloning the repo.
So I do that or you have any suggestion.

@jsvigneshkanna
Copy link
Owner

@harsh0620
You can create new branch from master after pulling latest changes, then add same code

Push those

@jsvigneshkanna
Copy link
Owner

@harsh0620 ,
Let me know if any help needed

@harsh0620
Copy link
Contributor Author

@jsvigneshkanna , the new PR has been generated.

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.

Add github forks/ stars widget in website homepage
2 participants