Skip to content

ENG-2129 Populate reference_content from CrossAppNode and roles from CrossAppNodeSchema - #1324

Open
maparent wants to merge 3 commits into
mainfrom
eng-2129-allow-to-populate-reference_content-from-crossappnode-and
Open

ENG-2129 Populate reference_content from CrossAppNode and roles from CrossAppNodeSchema#1324
maparent wants to merge 3 commits into
mainfrom
eng-2129-allow-to-populate-reference_content-from-crossappnode-and

Conversation

@maparent

@maparent maparent commented Aug 21, 2026

Copy link
Copy Markdown
Collaborator

@linear-code

linear-code Bot commented Aug 21, 2026

Copy link
Copy Markdown

ENG-2129

@vercel

vercel Bot commented Aug 21, 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 22, 2026 4:45pm

Request Review

@supabase

supabase Bot commented Aug 21, 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 ↗︎.

@maparent
maparent force-pushed the eng-2129-allow-to-populate-reference_content-from-crossappnode-and branch from a90808f to cca9575 Compare August 21, 2026 12:38
@maparent
maparent force-pushed the eng-2146-bulk-convert-some-db-structures-to-crossapp-structures-as branch 2 times, most recently from 34d0502 to dbd71ca Compare August 21, 2026 12:47
Base automatically changed from eng-2146-bulk-convert-some-db-structures-to-crossapp-structures-as to main August 21, 2026 12:52
@maparent
maparent force-pushed the eng-2129-allow-to-populate-reference_content-from-crossappnode-and branch from cca9575 to 2269c8c Compare August 21, 2026 12:55
@maparent
maparent force-pushed the eng-2129-allow-to-populate-reference_content-from-crossappnode-and branch from 2269c8c to 0684aad Compare August 21, 2026 13:06
@maparent
maparent force-pushed the eng-2129-allow-to-populate-reference_content-from-crossappnode-and branch 2 times, most recently from 388295f to 68b7d27 Compare August 21, 2026 20:40
@maparent
maparent marked this pull request as ready for review August 21, 2026 20:50

@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 2 potential issues.

View 1 additional finding in Devin Review.

Open in Devin Review

Comment thread packages/database/src/lib/sharedNodes.ts Outdated
Comment thread packages/database/src/lib/crossAppConverters.ts Outdated
@maparent
maparent force-pushed the eng-2129-allow-to-populate-reference_content-from-crossappnode-and branch from 68b7d27 to b98a419 Compare August 22, 2026 00:46
@maparent
maparent requested review from mdroidian and sid597 August 22, 2026 00:47
Comment thread packages/database/src/lib/sharedNodes.ts
@maparent
maparent force-pushed the eng-2129-allow-to-populate-reference_content-from-crossappnode-and branch from b98a419 to a998dc7 Compare August 22, 2026 16:30
@maparent
maparent force-pushed the eng-2129-allow-to-populate-reference_content-from-crossappnode-and branch from a998dc7 to ab1b460 Compare August 22, 2026 16:40
@maparent
maparent force-pushed the eng-2129-allow-to-populate-reference_content-from-crossappnode-and branch from ab1b460 to 03486e7 Compare August 22, 2026 16:43
@maparent

Copy link
Copy Markdown
Collaborator Author

Got Claude to add tests, which led to a correction.

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