Skip to content

[pull] main from tldraw:main#125

Merged
pull[bot] merged 2 commits intocode:mainfrom
tldraw:main
Aug 6, 2025
Merged

[pull] main from tldraw:main#125
pull[bot] merged 2 commits intocode:mainfrom
tldraw:main

Conversation

@pull
Copy link
Copy Markdown

@pull pull Bot commented Aug 6, 2025

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.3)

Can you help keep this open source service alive? 💖 Please sponsor : )

mootari and others added 2 commits August 6, 2025 09:27
Fixes #6547

Modifies `TLSocketRoom` to hold onto `onDataChange` and
`onPresenceChange`, and to pass them on whenever `TLSyncRoom` is
reinstantiated internally.

### Change type

- [x] `bugfix`
- [ ] `improvement`
- [ ] `feature`
- [ ] `api`
- [ ] `other`

### Test plan

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

### Release notes

- TBD
My understanding is that were were setting a property called `source`
when sending over the events. Since there was no other source set then
GA4 took this as the traffic source attribution as well.

So the problem is not actually using UTM tags for internal navigation.
The problem was that were we using the same name of the property that
GA4 uses for traffic attribution.

### Change type

- [x] `bugfix`
@pull pull Bot locked and limited conversation to collaborators Aug 6, 2025
@pull pull Bot added the ⤵️ pull label Aug 6, 2025
@pull pull Bot merged commit 67f438b into code:main Aug 6, 2025
@pull pull Bot requested a deployment to deploy-staging August 6, 2025 09:13 In progress
@pull pull Bot had a problem deploying to deploy-staging August 6, 2025 09:13 Error
@pull pull Bot had a problem deploying to vsce publish August 6, 2025 09:13 Failure
@pull pull Bot had a problem deploying to deploy-production August 6, 2025 09:13 Failure
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants