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

Simplify Polling and make the background instantly available #26

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

Conversation

ad-chaos
Copy link
Contributor

@ad-chaos ad-chaos commented Oct 9, 2023

This makes it so that the background doesn't have to wait on the network request to complete

@vercel
Copy link

vercel bot commented Oct 9, 2023

@ad-chaos is attempting to deploy a commit to the ACM_PESUECC Team on Vercel.

To accomplish this, @ad-chaos needs to request access to the Team.

Afterwards, an owner of the Team is required to accept their membership request.

If you're already a member of the respective Vercel Team, make sure that your Personal Vercel Account is connected to your GitHub account.

Copy link
Contributor

@anuragrao04 anuragrao04 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey, I got your code to local and found a few problems, check them out

.catch((e) => {
throw new Error(e.status, "Reddy Anna Is Not Talking");
});

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add leaderboard.sort((a, b) => b.Current_bounty - a.Current_bounty) here. The leaderboard isn't sorted from backend

setInterval(async () => {
await fetchLeaderboardData();
setInterval(() => {
invalidate("leaderboard");
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Polling doesn't work on local for me. Have you verified this works?

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.

2 participants