Skip to content

ENG-2064 Update Obsidian installaiton instruction#1246

Merged
trangdoan982 merged 4 commits into
mainfrom
eng-2064-update-obsidian-installaiton-instruction
Jul 24, 2026
Merged

ENG-2064 Update Obsidian installaiton instruction#1246
trangdoan982 merged 4 commits into
mainfrom
eng-2064-update-obsidian-installaiton-instruction

Conversation

@trangdoan982

@trangdoan982 trangdoan982 commented Jul 24, 2026

Copy link
Copy Markdown
Member

Summary

Rewrite the installation instruction to reflect that we're now LIVEEE

  • Reorders steps to BRAT → Datacore → Discourse Graphs
  • Folds the standalone "Requirements" bullet into an intro sentence that states the order up front.
  • Numbers the three install sections so they read as one sequence rather than independent options.
  • Standardizes "DataCore" → "Datacore" and applies sentence case to non-product headings.

Open question

Step 3 installs Discourse Graphs from Obsidian's community plugin browser. If DG isn't published there yet, that step should stay on the BRAT path (discourse-graph-obsidian repo) instead. Please confirm before merging.

🤖 Generated with Claude Code


Open in Devin Review

Reorder the install steps to follow the dependency chain
(BRAT → Datacore → Discourse Graphs) so the required Datacore
plugin is installed before Discourse Graphs.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@linear-code

linear-code Bot commented Jul 24, 2026

Copy link
Copy Markdown

ENG-2064

@supabase

supabase Bot commented Jul 24, 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 ↗︎.

@trangdoan982
trangdoan982 requested a review from jsmorabito July 24, 2026 13:15

@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.

Open in Devin Review

Comment thread apps/obsidian/README.md

@jsmorabito jsmorabito left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Datacore is now available in Obsidian's community plugin browser, so
drop the BRAT flow entirely. Both the website installation page and the
plugin README now install Datacore, then Discourse Graphs, directly from
Community plugins.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@vercel

vercel Bot commented Jul 24, 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, Comment Jul 24, 2026 9:50pm

Request Review

Comment thread apps/obsidian/README.md Outdated
Comment thread apps/website/content/obsidian/welcome/installation.md Outdated
@mdroidian

Copy link
Copy Markdown
Member

The graphite comments are worth addressing

Co-authored-by: graphite-app[bot] <96075541+graphite-app[bot]@users.noreply.github.com>
@jsmorabito

Copy link
Copy Markdown
Collaborator

The graphite comments are worth addressing

I clicked apply suggestion on the change, as I agree with its suggestion, but I dont see the comment updated. Are you able to tell if the commit went through?

Collapse the stray double blank line in installation.md so Prettier
passes, and update the README search term to "Discourse Graphs" to match
the website doc.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@trangdoan982
trangdoan982 merged commit e944c97 into main Jul 24, 2026
8 checks passed
@trangdoan982
trangdoan982 deleted the eng-2064-update-obsidian-installaiton-instruction branch July 24, 2026 21:52
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.

3 participants