delete old todo comment (#3777)

follow up to #3695 

### Change Type

<!--  Please select a 'Scope' label ️ -->

- [ ] `sdk` — Changes the tldraw SDK
- [ ] `dotcom` — Changes the tldraw.com web app
- [ ] `docs` — Changes to the documentation, examples, or templates.
- [ ] `vs code` — Changes to the vscode plugin
- [x] `internal` — Does not affect user-facing stuff

<!--  Please select a 'Type' label ️ -->

- [ ] `bugfix` — Bug fix
- [ ] `feature` — New feature
- [ ] `improvement` — Improving existing features
- [ ] `chore` — Updating dependencies, other boring stuff
- [ ] `galaxy brain` — Architectural changes
- [ ] `tests` — Changes to any test code
- [ ] `tools` — Changes to infrastructure, CI, internal scripts,
debugging tools, etc.
- [x] `dunno` — I don't know


### Test Plan

1. Add a step-by-step description of how to test your PR here.
2.

- [ ] Unit Tests
- [ ] End to end tests

### Release Notes

- Add a brief release note for your PR here.
This commit is contained in:
David Sheldrick 2024-05-20 10:13:52 +01:00 committed by GitHub
parent 4adbc76e2d
commit 3a96d040c3
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -424,9 +424,8 @@ export class TLSyncClient<R extends UnknownRecord, S extends Store<R> = Store<R>
private pushPresence(nextPresence: R | null) {
// make sure we push any document changes first
// TODO: need to send presence changes in the same push request as document changes
// in order to not get into weird states
this.store._flushHistory()
if (!this.isConnectedToRoom) {
// if we're offline, don't do anything
return