Skip to content

ENG-1225: Discourse node migration simple settings#696

Merged
sid597 merged 10 commits intomainfrom
eng-1225-discourse-node-migrate-settings-prod
Feb 24, 2026
Merged

ENG-1225: Discourse node migration simple settings#696
sid597 merged 10 commits intomainfrom
eng-1225-discourse-node-migrate-settings-prod

Conversation

@sid597
Copy link
Collaborator

@sid597 sid597 commented Jan 14, 2026

https://www.loom.com/share/9aa6f5f8815e41de92f21e14f7f5f855

demo video

changes done to following settings

"Key Image" — DiscourseNodeCanvasSettings.tsx

"Embedding Block Ref" — DiscourseNodeSuggestiveRules.tsx

"First Child" — DiscourseNodeSuggestiveRules.tsx

"Description" — NodeConfig.tsx

"Shortcut" — NodeConfig.tsx

"Tag" — NodeConfig.tsx

"Graph Overview" — NodeConfig.tsx

ENG-1225: Discourse node migration

Migrate personal settings text inputs

remove unnecessary checks zod takes care

migrate attributes overlay and suggestive rules


Open with Devin

Summary by CodeRabbit

  • Refactor
    • Migrated all settings storage from global to personal user-scoped configuration
    • Added real-time validation and error message feedback for node configuration settings, including tags and format specifications
    • Refactored component interfaces to reduce dependencies and improve maintainability
    • Standardized keyboard shortcut configuration across menu trigger features

✏️ Tip: You can customize this high-level summary in your review settings.

@linear
Copy link

linear bot commented Jan 14, 2026

@supabase
Copy link

supabase bot commented Jan 14, 2026

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 ↗︎.

@sid597 sid597 changed the title ENG-1225: Discourse node migration ENG-1225: Discourse node migration simple settings Jan 14, 2026
@sid597 sid597 marked this pull request as ready for review January 14, 2026 09:02
@sid597 sid597 force-pushed the eng-1225-discourse-node-migrate-settings-prod branch from 3ac2996 to d549f8d Compare January 17, 2026 18:16
@sid597 sid597 force-pushed the eng-1273-migrate-all-small-personal-settings branch from a82cfaf to bc0de10 Compare January 17, 2026 18:16
@sid597 sid597 force-pushed the eng-1225-discourse-node-migrate-settings-prod branch from d549f8d to 78b7dbb Compare January 18, 2026 05:25
@sid597 sid597 force-pushed the eng-1273-migrate-all-small-personal-settings branch from bc0de10 to 57abc42 Compare January 18, 2026 05:25
@sid597 sid597 force-pushed the eng-1225-discourse-node-migrate-settings-prod branch from 78b7dbb to d66fa03 Compare January 19, 2026 04:51
@sid597 sid597 force-pushed the eng-1273-migrate-all-small-personal-settings branch from 57abc42 to 7ead54e Compare January 19, 2026 04:51
@sid597 sid597 force-pushed the eng-1225-discourse-node-migrate-settings-prod branch from d66fa03 to cb22590 Compare January 19, 2026 05:01
@sid597 sid597 force-pushed the eng-1273-migrate-all-small-personal-settings branch from 490810f to 8dc5276 Compare January 19, 2026 06:03
@sid597 sid597 force-pushed the eng-1225-discourse-node-migrate-settings-prod branch from cb22590 to 78abef0 Compare January 19, 2026 06:03
@sid597 sid597 force-pushed the eng-1273-migrate-all-small-personal-settings branch from 8dc5276 to 8286d6b Compare January 19, 2026 06:34
@sid597 sid597 force-pushed the eng-1225-discourse-node-migrate-settings-prod branch from 78abef0 to 3966245 Compare January 19, 2026 06:34
This was referenced Jan 19, 2026
@sid597 sid597 force-pushed the eng-1273-migrate-all-small-personal-settings branch from 8286d6b to 78e8311 Compare January 20, 2026 16:06
@sid597 sid597 force-pushed the eng-1225-discourse-node-migrate-settings-prod branch from 3966245 to 7015148 Compare January 20, 2026 16:06
@sid597 sid597 changed the base branch from eng-1273-migrate-all-small-personal-settings to graphite-base/696 January 28, 2026 21:16
@sid597 sid597 force-pushed the eng-1225-discourse-node-migrate-settings-prod branch from 7015148 to ecdee21 Compare January 29, 2026 11:34
@sid597 sid597 force-pushed the graphite-base/696 branch from 78e8311 to 938f6aa Compare January 29, 2026 11:34
@sid597 sid597 changed the base branch from graphite-base/696 to eng-1273-migrate-all-small-personal-settings January 29, 2026 11:34
devin-ai-integration[bot]

This comment was marked as resolved.

@sid597 sid597 force-pushed the eng-1225-discourse-node-migrate-settings-prod branch from ecdee21 to 96f4575 Compare January 30, 2026 06:10
devin-ai-integration[bot]

This comment was marked as resolved.

@sid597 sid597 force-pushed the eng-1225-discourse-node-migrate-settings-prod branch from 7d6f127 to 26971c9 Compare February 23, 2026 10:26
@sid597
Copy link
Collaborator Author

sid597 commented Feb 23, 2026

@sid597 sid597 requested a review from mdroidian February 23, 2026 12:57
@sid597 sid597 force-pushed the eng-1225-discourse-node-migrate-settings-prod branch from b74f185 to 40b60a1 Compare February 23, 2026 16:40
Copy link
Contributor

@mdroidian mdroidian left a comment

Choose a reason for hiding this comment

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

a few minor changes and comments

@sid597 sid597 force-pushed the eng-1225-discourse-node-migrate-settings-prod branch from a266355 to 63ba316 Compare February 24, 2026 05:48
@sid597 sid597 merged commit 277aca6 into main Feb 24, 2026
8 checks passed
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