Skip to content

ENG-1563: Add Datacore dependency callout in settings#912

Open
trangdoan982 wants to merge 2 commits intomainfrom
eng-1563-user-is-able-to-install-dg-plugin-without-being-notified
Open

ENG-1563: Add Datacore dependency callout in settings#912
trangdoan982 wants to merge 2 commits intomainfrom
eng-1563-user-is-able-to-install-dg-plugin-without-being-notified

Conversation

@trangdoan982
Copy link
Collaborator

@trangdoan982 trangdoan982 commented Mar 25, 2026

Summary

  • Adds a warning callout banner at the top of the Discourse Graphs plugin settings when Datacore is not installed/enabled
  • Includes a link to install Datacore directly from the callout (obsidian://show-plugin?id=datacore)
  • Exports AppWithPlugins type from QueryEngine.ts for reuse

Test plan

https://www.loom.com/share/e993c36c02ef47dcbd98b4dbaff23eae

  • Open Obsidian without Datacore installed — verify warning callout appears at top of DG settings
  • Click "Install Datacore" link — verify it opens the Datacore plugin page
  • Install/enable Datacore, reopen settings — verify callout disappears

🤖 Generated with Claude Code


Open with Devin

@linear
Copy link

linear bot commented Mar 25, 2026

@supabase
Copy link

supabase bot commented Mar 25, 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 ↗︎.

devin-ai-integration[bot]

This comment was marked as resolved.

Shows a warning banner at the top of the settings page when the
Datacore plugin is not installed/enabled, with a link to install it.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@trangdoan982 trangdoan982 force-pushed the eng-1563-user-is-able-to-install-dg-plugin-without-being-notified branch from 9e016f8 to a33e97b Compare March 25, 2026 21:10
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.

1 participant