Skip to content

ENG-2126 Create a is_relation computed column on Concept - #1309

Merged
maparent merged 2 commits into
mainfrom
eng-2126-create-a-is_relation-computed-column-on-concept
Aug 21, 2026
Merged

ENG-2126 Create a is_relation computed column on Concept#1309
maparent merged 2 commits into
mainfrom
eng-2126-create-a-is_relation-computed-column-on-concept

Conversation

@maparent

@maparent maparent commented Aug 19, 2026

Copy link
Copy Markdown
Collaborator

@linear-code

linear-code Bot commented Aug 19, 2026

Copy link
Copy Markdown

ENG-2126

@vercel

vercel Bot commented Aug 19, 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 20, 2026 5:33pm

Request Review

@supabase

supabase Bot commented Aug 19, 2026

Copy link
Copy Markdown

Updates to Preview Branch (eng-2126-create-a-is_relation-computed-column-on-concept) ↗︎

Deployments Status Updated
Database Thu, 20 Aug 2026 17:32:22 UTC
Services Thu, 20 Aug 2026 17:32:22 UTC
APIs Thu, 20 Aug 2026 17:32:22 UTC

Tasks are run on every commit but only new migration files are pushed.
Close and reopen this PR if you want to apply changes from existing seed or migration files.

Tasks Status Updated
Configurations Thu, 20 Aug 2026 17:32:23 UTC
Migrations Thu, 20 Aug 2026 17:32:23 UTC
Seeding Thu, 20 Aug 2026 17:32:23 UTC
Edge Functions Thu, 20 Aug 2026 17:32:25 UTC

View logs for this Workflow Run ↗︎.
Learn more about Supabase for Git ↗︎.

@maparent
maparent force-pushed the eng-2126-create-a-is_relation-computed-column-on-concept branch from 6114637 to a441a33 Compare August 19, 2026 03:01
@maparent
maparent force-pushed the eng-2126-create-a-is_relation-computed-column-on-concept branch from a441a33 to b48b20f Compare August 19, 2026 14:34
@maparent
maparent marked this pull request as ready for review August 19, 2026 14:36
@maparent
maparent requested a review from mdroidian August 19, 2026 14:37
Comment thread packages/database/supabase/schemas/concept.sql

@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 2 additional findings in Devin Review.

Open in Devin Review

Comment thread packages/database/supabase/migrations/20260819023727_is_relation_column.sql Outdated

@mdroidian mdroidian left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Let's address the graphite/devon comments

@maparent
maparent requested a review from mdroidian August 19, 2026 21:54
@maparent
maparent removed the request for review from mdroidian August 19, 2026 23:17
@maparent
maparent force-pushed the eng-2126-create-a-is_relation-computed-column-on-concept branch from e4ed637 to 058eeeb Compare August 20, 2026 00:37
@maparent
maparent force-pushed the eng-2126-create-a-is_relation-computed-column-on-concept branch from 058eeeb to f49d19a Compare August 20, 2026 00:40
@maparent
maparent requested review from mdroidian and removed request for mdroidian August 20, 2026 00:47
@maparent
maparent force-pushed the eng-2126-create-a-is_relation-computed-column-on-concept branch from f49d19a to f42f667 Compare August 20, 2026 17:31
@maparent

Copy link
Copy Markdown
Collaborator Author

Rewritten both is_relation and arity with triggers with Claude help.

@maparent
maparent requested a review from mdroidian August 20, 2026 17:35
@maparent

Copy link
Copy Markdown
Collaborator Author

and updated the loom.

@maparent
maparent merged commit ff7ccc4 into main Aug 21, 2026
11 checks passed
@maparent
maparent deleted the eng-2126-create-a-is_relation-computed-column-on-concept branch August 21, 2026 12:40
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.

2 participants