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.

Sign up for GitHub

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

Jump to bottom

Display time remaining once voted and place bet async in backend to reduce loader time shown to users #373

Open
Tracked by #369
Natasha-GB opened this issue Aug 29, 2024 · 3 comments · May be fixed by #403
Open
Tracked by #369

Display time remaining once voted and place bet async in backend to reduce loader time shown to users #373

Natasha-GB opened this issue Aug 29, 2024 · 3 comments · May be fixed by #403

Comments

@Natasha-GB
Copy link
Contributor

Natasha-GB commented Aug 29, 2024 •
edited
Loading

Currently once you vote, you see the loader for 5-6 seconds - this is because the bet is being placed then. And post that, the “You have staked x tokens..” screen comes

What we want is once they vote, the staked token messaging screen comes right away and the bet placement takes place in backend asynchronously
@Natasha-GB Natasha-GB mentioned this issue Aug 29, 2024
Open
4 tasks
@harshita-srivastava-yral
Copy link
Contributor

  • @Veercodeprog Study the optimistic UI and try to get the time remaining from client side instead of sending it to backend and displaying the loading.
  • Similarly, do the same for Wallet balance, keep the balance stored at client side, keep calculating and sending to backend. Once backend has insufficient balance it will send error to client which we can display.
  • User Current time - Created at = Time left for slot to end

@harshita-srivastava-yral
Copy link
Contributor

To be started today

veer-pratap-yral added a commit that referenced this issue Sep 11, 2024
@veer-pratap-yral
Display remaining time after vote and place bet async to reduce loade…
e57f103 
…r time (#373)
@veer-pratap-yral veer-pratap-yral linked a pull request Sep 11, 2024 that will close this issue
Open
@harshita-srivastava-yral
Copy link
Contributor

  • @harshitasrivastav28 to look into the preview link and give go ahead for the same

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@harshita-srivastava-yral @Natasha-GB @veer-pratap-yral