-
Notifications
You must be signed in to change notification settings - Fork 3
Nodetag documentation #433
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
joelchan
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
reviewed and approve content, and code looks like it will merge but i'm not yet set up to test so can't verify. maybe final sign-off from @mdroidian on that front?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR is being reviewed by Cursor Bugbot
Details
You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.
To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.
|
@mattakamatsu @joelchan I made some changes:
|
mdroidian
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should change "Tagging Potential Nodes" to "Tagging Candidate Nodes." to be consistent and avoid user confusion, but looks good other than that!
You can view it here: https://discourse-graph-git-nodetag-documentation-discourse-graphs.vercel.app/docs/roam/tagging-candidate-nodes
|
|
||
| ## Candidate node styling | ||
|
|
||
| Candidate node styling is currently controlled in roam/css. You can borrow our node styling here: [template-lab CSS page](https://roamresearch.com/#/app/template-lab/page/X8V4gy32s) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| published: true | ||
| --- | ||
|
|
||
| ## Purpose - quickly bookmark notes that could later become formal discourse nodes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should discuss and agree on a consistent capitalization convention for terms like Discourse Graphs, Discourse Nodes, Candidate Nodes, node types (e.g., Claim, Evidence, etc.), etc.
I added this as part of a GitHub issue: #7 but don't think it ever made it into Linear.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
connecting to discord poll (poll still open)
* initial node tagging documentation page with gifs * add instructions in tagging-candidate-nodes documentation * JC edits 1 on tagging candidate nodes docs * add metadata * Format and Remove Title * move images and update links * address comments about page title, simplify callout styling * bold to italics for callout text --------- Co-authored-by: Michael Gartner <mclicks@gmail.com>
Add nodetag documentation; list it between "Creating nodes" and "creating relations"