-
Notifications
You must be signed in to change notification settings - Fork 34
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
Refactor Credits Page #193 #203
Conversation
Merge Staging to Prod : I APPROVE OF THIS MWAAHHHAHA
chore : Changed Err Message for Debug /hustle
chore : Changed Err Message for Debug /hustle
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.
@Nexusrex18 You dont need a cron job for syncing the contributors, it is a simple API call to the github api. You can call it everytime the page is loaded, with a loader
@SkySingh04 but is pr ka kya kru bhaiya hatana hai ya naya pr daalu |
@SkySingh04 bhaiya thik kar diya wo ek baar check kar lena :D |
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.
LGTM
Fixes #193
Changes Made
Video preview
Screen.Recording.2025-01-25.171637.mp4
Comments:
@SkySingh04 bhaiya , this issue is most probably arising due to existing hydration error of website which i cant find out in my localhost preview and also on reloading the page again it fixes itself but im trying to find the solution for it but i will fix it as i find the solution.
2.
For syncing the github contributors api i need cron job but maine @2004yash se pucha tha to usne bataya ki production server ko github easily allow nhi krte data fetching ke liye but mai ye issue fix kr dunga baad me naya issue create krke tab tak weekly mjhe POST request manually run krni pdegi.