Skip to content

ENG-2165 Refactor Roam Sync to use crossAppNodes - #1333

Draft
maparent wants to merge 4 commits into
mainfrom
ENG-2165-Refactor-Roam-Sync-to-use-crossAppNodes
Draft

ENG-2165 Refactor Roam Sync to use crossAppNodes#1333
maparent wants to merge 4 commits into
mainfrom
ENG-2165-Refactor-Roam-Sync-to-use-crossAppNodes

Conversation

@maparent

@maparent maparent commented Aug 23, 2026

Copy link
Copy Markdown
Collaborator

@linear-code

linear-code Bot commented Aug 23, 2026

Copy link
Copy Markdown

ENG-2165

@vercel

vercel Bot commented Aug 23, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
discourse-graph Ready Ready Preview Aug 23, 2026 1:15pm

Request Review

@graphite-app

graphite-app Bot commented Aug 23, 2026

Copy link
Copy Markdown
Contributor

PR size/scope check

This PR is over our review-size guideline.

  • Recommended: ~200 lines changed
  • Acceptable limit: up to 400 lines when well-scoped/self-contained
  • Preferred file count: fewer than 5 files

Please split this into smaller PRs unless there is a clear reason the changes need to land together.

If keeping it as one PR, please add a brief justification covering:

  • What single problem this PR solves
  • Why the files/changes are coupled

@supabase

supabase Bot commented Aug 23, 2026

Copy link
Copy Markdown

This pull request has been ignored for the connected project zytfjzqyijgagqxrzbmz because there are no changes detected in packages/database/supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Devin Review found 1 potential issue.

View 1 additional finding in Devin Review. (Configure)

Open in Devin Review

Comment thread apps/roam/src/utils/roamToCrossAppConverters.ts
@maparent
maparent marked this pull request as draft August 23, 2026 13:07
@maparent
maparent force-pushed the ENG-2165-Refactor-Roam-Sync-to-use-crossAppNodes branch from 06bc853 to 9ca6cde Compare August 23, 2026 13:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant