-
Notifications
You must be signed in to change notification settings - Fork 27
fix: Enable forecast checkbox for multiple choice questions #4056
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.
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
Conversation
Remove the restriction that prevented the 'include your current forecast' checkbox from appearing on multiple choice questions. The feature was already fully supported by the backend and rendering components but was artificially disabled in the form logic. Fixes #4053 Co-authored-by: Sylvain <[email protected]>
📝 WalkthroughWalkthroughThe changes remove restrictions that prevented forecast inclusion for MultipleChoice questions. Two components were updated: an early return condition was eliminated, and the forecast filtering logic was relaxed to allow forecasts across all question types when a post contains an active latest forecast. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Poem
🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
📜 Recent review detailsConfiguration used: Organization UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (2)
💤 Files with no reviewable changes (1)
🧰 Additional context used🧠 Learnings (1)📓 Common learnings⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (4)
🔇 Additional comments (1)
✏️ Tip: You can disable this entire section by setting Comment |
🧹 Preview Environment Cleaned UpThe preview environment for this PR has been destroyed.
Cleanup triggered by PR close at 2026-01-23T15:39:17Z |
|
does the prediction in comment look right? @SylvainChevalier @aseckin
|
|
@cemreinanc not perfect but better than not having it for sure! |
Sure! Then merging it as is now. @SylvainChevalier Open a new issue if @aseckin updates the design in the future |

Remove the restriction that prevented the 'include your current forecast' checkbox from appearing on multiple choice questions. The feature was already fully supported by the backend and rendering components but was artificially disabled in the form logic.
Fixes #4053
Generated with Claude Code
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.