Skip to content

ENG-1412: Documentations for sync relations#901

Open
trangdoan982 wants to merge 6 commits intomainfrom
eng-1412-documentations-for-sync-relations
Open

ENG-1412: Documentations for sync relations#901
trangdoan982 wants to merge 6 commits intomainfrom
eng-1412-documentations-for-sync-relations

Conversation

@trangdoan982
Copy link
Collaborator

@trangdoan982 trangdoan982 commented Mar 18, 2026

@linear
Copy link

linear bot commented Mar 18, 2026

@supabase
Copy link

supabase bot commented Mar 18, 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
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: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 3 additional findings.

Open in Devin Review

@trangdoan982 trangdoan982 force-pushed the eng-1412-documentations-for-sync-relations branch from c4a73ec to 7ac13c9 Compare March 18, 2026 19:34
devin-ai-integration[bot]

This comment was marked as resolved.

devin-ai-integration[bot]

This comment was marked as resolved.

graphite-app[bot]

This comment was marked as resolved.

trangdoan982 and others added 3 commits March 18, 2026 15:03
Co-authored-by: devin-ai-integration[bot] <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Co-authored-by: graphite-app[bot] <96075541+graphite-app[bot]@users.noreply.github.com>
Copy link
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 new potential issue.

View 5 additional findings in Devin Review.

Open in Devin Review

- **"Please sync the node first"** — The node hasn't been synced yet. Wait for automatic sync or trigger a manual sync
- **"Please sync the node first"** — The node hasn't been synced yet. Wait for automatic sync or trigger a manual sync via the command palette using **"Discourse Graph: Sync discourse nodes to Supabase"**.

![Sync command](/docs/obsidian/sync-command.png)
Copy link
Contributor

Choose a reason for hiding this comment

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

🟡 Unindented image breaks the troubleshooting markdown list into two separate lists

The ![Sync command] image at line 145 is not indented under the preceding list item and is surrounded by blank lines. In standard markdown, this breaks the bulleted list into two separate lists — items before the image become one list, and items after it start a new list. Compare with line 150 where ![Show frontmatter] is correctly indented with 2 spaces to remain part of the list item above it.

Suggested change
![Sync command](/docs/obsidian/sync-command.png)
![Sync command](/docs/obsidian/sync-command.png)
Open in Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.


- **"You are not a member of any groups"** — You need to be added to a group before you can import nodes. Contact your team administrator
- **No importable nodes found** — Either no nodes have been published to your groups, or you have already imported all available nodes
- If you want to see more information about imported nodes, you can unhide frontmatter information in the setting panel
Copy link
Collaborator

Choose a reason for hiding this comment

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

Nit: unhide is inelegant. I would say:
"The frontmatter fields related to sync are normally hidden. You can choose to display them in the setting panel."

Otherwise lgtm.

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