powerhouse-inc / connect

https://document-model-electron.vercel.app
3 stars 2 forks source link

Update Icon behavior for Syncing feedback. #382

Open CallmeT-ty opened 1 month ago

CallmeT-ty commented 1 month ago

This is a symptom of the initial sync flow. It's sending a pushUpdates for each file. Connect is using a single queue so it's processing them in sequence. Ideally, we would simply don't send the pushUpdates as the updated strands we are sending just came from Switchboard. In terms of the UI, maybe the icons should start as Syncing instead of success then the folders icon would gradually turn to synced

Got it. That makes sense. I'll think a bit more about how the algorithm should keep track of remote sync statuses... ideally it would translate naturally to peer-to-peer connections too.

CallmeT-ty commented 3 weeks ago

Let's deprioritize the issue untill we have the latest input from @acaldas on the websockets progress.

CallmeT-ty commented 2 weeks ago

In theory: We load the operations, we pply all operation to document drive. Applying to operations causes the syncing? If you create a document during the sync it will reshuffle and send it to switchboard (Local conflict)

Easiest way to look into this issue: Syncstatus messages vs. UI problem.