-
Notifications
You must be signed in to change notification settings - Fork 27
Reorder fields in question creation forms #4288
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
- Move "Group Variable" field below "Categories" and above "Subquestions" in group question editor - Move time fields (Close Time, Resolving Time, Open Time, CP Reveal Time) below NumericQuestionInput in continuous question editor Co-authored-by: Luke Sabor <lsabor@users.noreply.github.com>
📝 WalkthroughWalkthroughForm fields in question and group editors are reordered per user request. The group variable field is repositioned below categories, and time-related date inputs are moved into a new Advanced Options section within the question form. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 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
🧪 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 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@front_end/src/app/`(main)/questions/components/question_form.tsx:
- Around line 889-916: The InputContainer for Open Time uses a hardcoded English
explanation string; replace explanation={"When this question will be open for
predictions."} with a translation call (e.g.
explanation={t("openTimeDescription")} or another appropriate i18n key) so it
uses the existing t(...) pattern; update the translation files to add the chosen
key if missing and keep the reference in the InputContainer props (look for the
InputContainer instance that wraps DateInput with name="open_time").
🧹 Preview Environment Cleaned UpThe preview environment for this PR has been destroyed.
Cleanup triggered by PR close at 2026-02-09T15:56:48Z |
ncarazon
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Fixes #3592
Changes
The field ordering now matches the requested layout as shown in the issue screenshots.
🤖 Generated with Claude Code
Summary by CodeRabbit