feat(app): enable click action for gauge widgets - #373
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (3)
WalkthroughThese changes enable click action support for gauge charts by updating the chart registry configuration to mark gauge as supporting click actions, wiring the click handler in the chart renderer component, and updating the corresponding test expectations. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 3 | ❌ 2❌ Failed checks (2 warnings)
✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
…ction feat(app): enable click action for gauge widgets



Summary
supportsClickAction: truein the gauge entry ofchart-registry.ts.onClick={handleEChartsClick}throughchart-renderer.tsxso the gauge's BaseChart receives ECharts click events.Closes #152
Test plan
app/src/lib/__tests__/chart-registry.test.tsupdated — 271 tests passCo-Authored-By: Claude Opus 4.6 (1M context) noreply@anthropic.com
Summary by CodeRabbit