Update useRoamStore to use updatePage instead of updateBlock#311
Update useRoamStore to use updatePage instead of updateBlock#311
Conversation
WalkthroughVersion updated to 1.36.1. In tldraw’s Roam integration, persistence switched from updating a block to updating a page, changing the API method and payload structure to write page-level props. Changes
Sequence Diagram(s)sequenceDiagram
autonumber
participant U as User
participant T as Tldraw UI
participant S as useRoamStore
participant R as Roam API
U->>T: Edit canvas/state
T->>S: Request persist
S->>R: updatePage({ page: { uid, props } })
R-->>S: Ack/Result
S-->>T: Persisted status
sequenceDiagram
autonumber
rect rgb(245,245,255)
note over S,R: Old (before)
S->>R: updateBlock({ block: { uid, props } })
end
rect rgb(245,255,245)
note over S,R: New (after)
S->>R: updatePage({ page: { uid, props } })
end
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches
🧪 Generate unit tests
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (2)
🔇 Additional comments (1)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
No description provided.