-
-
Notifications
You must be signed in to change notification settings - Fork 22.3k
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
All stats card and repo features are down. Error cannot fetch user is shown on all cards including your readme #bug #51
Comments
Same here, too. |
Probably hit the API limit.... @ApurvShah007 |
@anuraghazra Is there any possible solution? I really like this feature and I am down to help fix this issue. @filiptronicek Thats probably what happened |
Github API limit is hit. it should be up in some time |
It's really a bummer that github api only allows 5k requests per hour. sorry guys i'll see what i can do to fix the issues. |
@anuraghazra That is a bummer, how do you plan to fix it? Will it fix itself? |
It will fix itself after a window of time, but i'm working on a proper fix so that it should not happen too often |
But it is just a temporary solution, I think there is just one solution: delete the Vercel project, don't let anyone use it, and force everyone to use their self-hosted version with their own PAT. |
@filiptronicek I was about to suggest the same thing, I am pretty sure everyone can host their own requests making it self-sufficient. |
I know, I host one myself, but I would make it mandatory, it isn't very wise to encourage people to use something, that doesn't work a lot of the time. This project just doesn't scale well, although it is an awesome tool! |
Yup i agree. i might migrate to doing that, but i'm currently experimenting with caching and collaborating with @statically.io to see what would work best. |
@filiptronicek and @anuraghazra am I supposed to deploy it on the repo that exists or should I make a new one? I am kind of a boomer sorry. |
you can fork this repo and click on the deploy button after that generate your github personal token and use put it on vercel |
I suggest you fork the repo, and set up Vercel with that repo because then you can always update your own |
Talking of the self-hosted version, any idea why I have an A++ ranking on the official site, but in my own instance, I have a ranking of just A+, nothing, that would anger me or something, but it still is weird. @anuraghazra |
@anuraghazra @filiptronicek do I allow it to install on all my repos? |
Yes, I would suggest you do, for future usage, but if you don't plan on using it, just install it on the one fork @ApurvShah007 |
@filiptronicek Well I plan to have the GitHub readme stats card and feature a few repos on my profile readme. IDK what I should do |
Just allow all repos then, will be easier @ApurvShah007 |
Another idea @anuraghazra, we could implement multiple PATs like this: We could handle 50K request with just 10 tokens, every hour |
Is it back up for everyone or did I set up my Vercel correctly? |
It is back up @ApurvShah007 |
Yup there is some issues with ranking system its currently biased, checkout #41 #49 for more info. |
Ok I still want to set up mine properly. I let it install on all my repos and the project is up on Vercel |
What are PATs? Can we do that? like each github personal token would have its own 5k limit? |
@filiptronicek @anuraghazra I feel like you should make a guide on how to set up the while thing. Specially for people like me |
PATs are personal access tokens, if we had more, we could do more requests |
Yup i'll do it tomorrow got 3 same questions today. |
Ok, so @anuraghazra |
@filiptronicek I don't mind being one user if you ever need it. Just let me know what to do. |
@filiptronicek @anuraghazra, count me in. |
Sounds great 😃 @ApurvShah007 @garvit-joshi thanks for the support, I highly appreciate it. Can you guys leave your Twitter so that I can contact you about this. |
@anuraghazra I don't have a twitter we can do LinkedIn or Insta too if that works |
@ApurvShah007 insta would work 😃 |
@filiptronicek on Twitter @anuraghazra |
@anuraghazra I followed you back in insta |
|
Can we not add a clause that fetches new updates every 24 hours and use the image obtained for whole day? Just a suggestion, I'm noob. |
@infinity-plus 24 hours Is a bit much, I think you would like the stats to be frequently updated. Currently I have 5min cache setup, so I'll fetch new data every 5min |
@anuraghazra I don't think anybody checks my git every 5mins tho, + this repo is getting popular |
@anuraghazra The live number of visitors is also dead. Not sure if I took it from your repo or not but do you have any idea? |
No there is no relation with that thing whatsoever. |
@anuraghazra |
Yes. It hit the rate limit too. |
@TheAshwanik , acc. to my understanding the issue has been fixed, PAT have been increased, I am using the original instance and it works fine. |
@garvit-joshi I was talking about the "live number of visitors" |
@TheAshwanik I would also like to deploy my own so I dont run into the same problem. It would be helpful if you can make a short tutorial on that. |
Will do.. |
@ApurvShah007 @TheAshwanik can you guys continue the discussions regarding visitor badge here: https://github.com/jwenjian/visitor-badge It's the official repo, please open an issue there. Since this repo does not have any connection with visitor badge I think it's not the right place to discuss it. Otherwise you can always hop into my discord server if you have any issues. Thanks! |
@anuraghazra Sorry about that. I would love a link to your discord. |
No description provided.
The text was updated successfully, but these errors were encountered: