Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Current version breaks Discourse Relations #88

Closed
mdroidian opened this issue May 28, 2023 · 3 comments
Closed

Current version breaks Discourse Relations #88

mdroidian opened this issue May 28, 2023 · 3 comments
Assignees
@dvargas92495

Comments

@mdroidian
Copy link
Collaborator

@dvargas92495

Steps to repo:

  • load https://samepage.network/releases/dvargas92495/roamjs-query-builder/
  • enable Discourse Graphs
  • go to roam/js/discourse-graph and preview any relation

image

2023-05-28.15-22-57.mp4
@dvargas92495 dvargas92495 self-assigned this May 29, 2023
@mdroidian
Copy link
Collaborator Author

While the previews are fixed, it looks like the relations are still not working. @dvargas92495

https://roamresearch.slack.com/archives/CNC4881EU/p1685876834972689

@mdroidian mdroidian reopened this Jun 4, 2023
@mdroidian
Copy link
Collaborator Author

Update:
I can query the relations, eg: "Evidence Opposes Claim"

It might only be the Discourse Context and Overlay that are affected.

@mdroidian
Copy link
Collaborator Author

mdroidian commented Jun 5, 2023 •
edited
Loading

It looks like all discourse nodes are being set as type: page-node

dvargas92495 added a commit that referenced this issue Jun 5, 2023
@dvargas92495
Fix Discourse Nodes default ordering (Closes #88)
c357238
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dvargas92495 dvargas92495

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mdroidian @dvargas92495