-
Notifications
You must be signed in to change notification settings - Fork 10
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
Better UX when server is unresponsive #73
Comments
In case you want to work on this, feel free to assign yourself. If not, we will wait for someone interested to comment and we can assign the issue to them : ) |
Hmm, I don't think users can assign themselves to issues unless they're added as collaborators on a repository. I would be willing to work on it later but at the moment I am looking into #68, could you assign me to that please? |
sure, got it! if you want to work on any issue, just leave a comment and we will assign it to you! |
can you leave a comment there so I can assign it to you? |
Would this behaviour solve this issue: |
that's correct ! maybe you can do a Alert/Snackbar with library for better UI : ) something like this https://mui.com/material-ui/react-snackbar/ |
Alrighty, can you assign this issue to me? |
Done! Good luck !! Feel free to ping us if you have any questions or want to discuss anything! |
One thing to also note is, we also added in an additional feature to prevent students from spamming votes all the time. If you have more questions about it @shubhbapna can probably talk more about this |
Describe the bug
Currently, the selected vote only changes on the frontend once the server has acknowledged the vote. This leads to bad UX if the server is slow.
Expected behavior
Ideally, there should be some sort of visual indication that the vote is being sent to the server, and if no acknowledgement is received after 10 seconds or so, an error should be displayed.
Additional context
This is particularly relevant until we are able to sort out the ongoing server load issues.
The text was updated successfully, but these errors were encountered: