-
-
Notifications
You must be signed in to change notification settings - Fork 566
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
Shopping List Autosync #3217
Comments
Yes, the shopping cart icon flashes slightly(it tries to quickly draw the refresh icon) in the selected 5s interval. |
Same issue here. The request gets sent to the server and returns 200 OK. If I add an Item, it will show up. |
ok, thanks for the feedback. I think this is a timing issue. I tried to optimize performance but probably broke something |
Tandoor Version
1.5.18
Setup
Docker / Docker-Compose
Reverse Proxy
Traefik
Other
Also reproduced in demo
Bug description
In short, I seem to have a problem with shopping list autosync- it does not work most of the time.
I have Tandoor deployed via docker-compose but this should be no matter, because I also was able to reproduce this on the tandoor demo.
When I set the autosync to 5 seconds(10 for the demo) I expect that when I mark an item as bought, that the item disappears from other devices where the shoppinglist is shared to after 5-20 seconds.
For me, this is not the case- sometimes an item is not synced for a long time, until the app is reopened or the shopping-list reopened.
I add an item on one session, it does not appear on the other session unless I refresh via the other browser. Same for removing the item via the green check-mark.
On the demo just now in the shopping list, I added an item, it appeared on the other session. Then I removed it in the first session, but it is still visible on the other session a good few minutes later.
If I press F5 in the other session, it disappears as expected.
This behaviour happens for me both on Firefox on PC with one or several sessions, aswell as between Sessions on Firefox(PC) and Safari(iOS).
Is it just me or is this known?
Relevant logs
The text was updated successfully, but these errors were encountered: