ENG-2120 Add documentation for bulk importing feature (Obsidian) - #1291
ENG-2120 Add documentation for bulk importing feature (Obsidian)#1291trangdoan982 wants to merge 2 commits into
Conversation
…ian) Document the "Bulk identify discourse nodes" command, which had no user-facing docs. Adds a new page under Obsidian advanced features covering the three-step wizard, the title-matching rules that are not visible in the UI, and what identification does and does not change. Also cross-links from the single-note "Convert into" flow.
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
|
@codex review |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: cea25c45ca
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Two claims in the page did not match the implementation:
- "Everything outside {content} is matched literally" — the matcher escapes
only [ ] ? . + , so ( ) * | ^ $ reach the RegExp as syntax. Replaced the
soft "may not match as expected" note with an explicit warning.
- "Notes with a node type already set are excluded" — the scan excludes only
IDs present in the configured node types, and handleBulkIdentify then
overwrites any stale nodeTypeId. Qualified the exclusion and documented the
overwrite.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
2 participants
Documents the Bulk identify discourse nodes Obsidian command, which shipped without user-facing docs.
Linear: ENG-2120
New page at
/docs/obsidian/advanced-features/bulk-identify-discourse-nodes, with four screenshots and a cross-link from Creating nodes.Beyond the three-step wizard, it documents what isn't discoverable from the UI:
nodeTypeIdare excludednodeTypeId— no rename, no move, no template, no bulk undoThat last point is the reason for the page: "identify" reads like "convert," and it isn't.
Verified on a local dev server: page and images render, sidebar order correct, all cross-links and anchors resolve, prettier clean.
Scope check
$scope-checkagainst ENG-2120 and the final diff.Done When: The ticket'sDone Whenis empty, so there is no stated boundary to check against. Scoped toProblem("feature no documentation") — docs only, no runtime code touched.Noted, not fixed
getDiscourseNodeFormatExpressionescapes[ ] ? . +but not( ) * |, so formats with parentheses silently mis-match. Documented as a "prefer simple prefixes" note; likely worth its own ticket.🤖 Generated with Claude Code