Prevent unnecessary fetching of readonly slugs

[MitjaBezensek]

Prevents unnecessary fetching of readonly slugs. We only need to fetch it if we don't have it yet.

There was also a weird issue with window.location.href sometimes returning encoded search params and sometimes decoded ones:

This then caused an additional fetch in the setInterval since the urls did not match.

Resolves https://github.com/tldraw/tldraw/issues/3661

Change Type

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

• [ ] bugfix — Bug fix
• [ ] feature — New feature
• [x] 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.
• [ ] dunno — I don't know

Test Plan

Opening a multiplayer room

1. Open a multiplayer room. You should only see one fetch for the readonly slug.

Sharing a local room

1. Open a local room.
2. Share it. You should only see one fetch for the readonly slug.

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

Release Notes

• Prevent unnecessary fetching of readonly slugs.

[vercel[bot]]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated (UTC)
examples	✅ Ready (Inspect)	Visit Preview	May 3, 2024 10:29am

1 Ignored Deployment

| Name | Status | Preview | Updated (UTC) | | :--- | :----- | :------ | :------ | | **tldraw-docs** | ⬜️ Ignored ([Inspect](https://vercel.com/tldraw/tldraw-docs/F1zriMGv933hmmaRDcPzgTvafrQG)) | [Visit Preview](https://tldraw-docs-git-mitja-readonly-fetch-tldraw.vercel.app) | May 3, 2024 10:29am |