-
Notifications
You must be signed in to change notification settings - Fork 39
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
Clap count in UI is increment while the update is rejected by server #57
Comments
@ColinEberhardt Let me know which solution is preferable. I can fix this issue. |
Any update on this one? I opened another issue probably related to this one due to the introduction of Here is the link: #69 Thanks for such a great job @ColinEberhardt ❤️ |
Thanks for the suggestions - this is a tricky issue, and one that always has an element of compromise. I've made a recent update that works as follows:
This does result in some behaviour that users might find unexpected:
|
@ColinEberhardt thanks for taking care of this! Amazing job! ❤️ |
Once a user clap for second time after page refresh, It increments the count in UI.
But the update api call does not increase the count as the clap limit exceeded for the user from that IP, it returns the same old value for clap count.
Expected:
If the clap limit exceeded for the user(source ip), disable the button on initialisation(page refresh)
Solutions:
The text was updated successfully, but these errors were encountered: