Skip to content

ENG-1695: Update Obsidian docs for create-node flows#1100

Open
trangdoan982 wants to merge 4 commits into
mainfrom
eng-1695-update-documentation-for-create-node-flows-after-merging-all
Open

ENG-1695: Update Obsidian docs for create-node flows#1100
trangdoan982 wants to merge 4 commits into
mainfrom
eng-1695-update-documentation-for-create-node-flows-after-merging-all

Conversation

@trangdoan982
Copy link
Copy Markdown
Member

@trangdoan982 trangdoan982 commented Jun 1, 2026

Summary

  • Rewrites Creating discourse nodes for current flows: hotkey picker, right-click, command palette, embedded image convert, file explorer/file menu convert, node tags, and canvas; documents modal behavior (search existing, relationship, insert backlink).
  • Updates Canvas discourse-node section: create via toolbar/modal, add existing via modal search, drag from file explorer and editor wikilink handles, convert text/image shapes; removes stale top-right node search UI.
  • Updates Node tags to mention existing-node selection and link to creating-discourse-nodes.
  • Adds local screenshots under apps/website/public/docs/obsidian/ (replaces Firebase-hosted images for create-node page).

Test plan

  • Open /docs/obsidian/core-features/creating-discourse-nodes locally and verify images and steps match plugin behavior
  • Open /docs/obsidian/core-features/canvas and confirm drag flows (file explorer + editor) and convert-shape steps
  • Open /docs/obsidian/core-features/node-tags and confirm cross-link works
  • Confirm no broken image paths under /docs/obsidian/*

Made with Cursor


Open in Devin Review

Refresh creating-discourse-nodes, canvas, and node-tags to match current
modal-based flows, file explorer convert, editor/file drag-to-canvas, and
embedded image conversion. Add local screenshots under public/docs/obsidian.

Co-authored-by: Cursor <cursoragent@cursor.com>
@linear-code
Copy link
Copy Markdown

linear-code Bot commented Jun 1, 2026

ENG-1695

@graphite-app
Copy link
Copy Markdown
Contributor

graphite-app Bot commented Jun 1, 2026

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
Copy link
Copy Markdown

supabase Bot commented Jun 1, 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 ↗︎.

Copy link
Copy Markdown
Contributor

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

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 apps/website/content/obsidian/core-features/creating-discourse-nodes.md Outdated
trangdoan982 and others added 2 commits June 1, 2026 11:56
Obsidian has no querying-discourse-graph content page; link to canvas instead.

Co-authored-by: Cursor <cursoragent@cursor.com>
Comment thread apps/website/content/obsidian/core-features/creating-discourse-nodes.md Outdated
Co-authored-by: Cursor <cursoragent@cursor.com>
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