H-6717: Dashboard config and refresh improvements#9085
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
PR SummaryMedium Risk Overview Manual configuration treats chart series as the source of truth: The document inference flow fixes author email property metadata to use Reviewed by Cursor Bugbot for commit 8a9711a. Bugbot is set up for automated code reviews on this repo. Configure here. |
…m-config.ts Co-authored-by: graphite-app[bot] <96075541+graphite-app[bot]@users.noreply.github.com>
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Want reviews to match your repository better? Bugbot Learning can learn team-specific rules from PR activity. A team admin can enable Learning in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 8a9711a. Configure here.
| } | ||
|
|
||
| if (!chartType || !chartData) { | ||
| if (!chartConfig) { |
There was a problem hiding this comment.
Maps require chart config now
Medium Severity
Position-based map items previously rendered from chartType and chart data alone via WorldMapRenderer, which does not use chartConfig. The new early chartConfig guard blocks that path, so map items that only have chartType set show “Missing chart configuration” instead of the map.
Additional Locations (1)
Reviewed by Cursor Bugbot for commit 8a9711a. Configure here.


🌟 What is the purpose of this PR?
Fix some bugs and improve behaviour related to the custom dashboard page:
Also fix a bug in the document inference flow where patching an email was referring to the wrong property key.
Pre-Merge Checklist 🚀
🚢 Has this modified a publishable library?
This PR:
📜 Does this require a change to the docs?
The changes in this PR:
🕸️ Does this require a change to the Turbo Graph?
The changes in this PR: