Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (5)
Included review availability: Your plan provides up to 2 included reviews per hour; 1 remains after this review. 📝 WalkthroughWalkthroughThe PR adds conditional question rendering to embedded views. It renders both outcome charts, preserves disabled annulled branches, forwards chart settings, adds conditional scrolling, and assigns conditional embed dimensions. ChangesConditional embed support
Priority: ➖ Normal Estimated code review effort: 3 (Moderate) | ~20 minutes Change: Bug fix Sequence Diagram(s)sequenceDiagram
participant EmbedQuestionPlot
participant EmbedConditionalPlot
participant ConditionalChart
EmbedQuestionPlot->>EmbedConditionalPlot: pass conditional post and theme
EmbedConditionalPlot->>ConditionalChart: render If Yes chart
EmbedConditionalPlot->>ConditionalChart: render If No chart
Merge Risk: ⚪ Minimal · up to Conditional embeds render both supported branches with no established unresolved behavior issue. 🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 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. A rabbit hops where forecasts meet, Comment |
🚀 Preview EnvironmentYour preview environment is ready!
Details
ℹ️ Preview Environment InfoIsolation:
Limitations:
Cleanup:
|
Conditional post embeds currently render a title and footer without either forecast branch because
EmbedQuestionPlothandles only individual questions and groups.Add a compact conditional renderer with the condition, outcome, and both If Yes / If No forecasts. It reuses
ConditionalChartfor existing forecast availability, CP visibility, resolution, and chart-theme behavior. Branches stack in narrow cards and sit side by side when space permits. Conditional embeds get a 420px default height; explicitly short embeds scroll their content while keeping the footer accessible. Social-preview sizing scales by height to avoid clipping the taller card.Validation:
embed_question_card.tsx. Whole-repository lint was obscured by Windows CRLF checkout errors.The country-map consumer will keep conditionals excluded until this renderer is released on Metaculus.
Summary by CodeRabbit
New Features
Tests