-
Notifications
You must be signed in to change notification settings - Fork 20
Implement A/B test for question creation button #3623
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
base: main
Are you sure you want to change the base?
Conversation
front_end/messages/en.json
Outdated
| "commentUnpinned": "Comment unpinned", | ||
| "finePrintDescription": "Optional: Use the fine print for any sort of lawyerly details which don't need to be prominently displayed.", | ||
| "createQuestion": "create question", | ||
| "submitAQuestion": "submit a question", |
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.
"Submit" and "Question" should be capitalized.
| const getCreateButtonConfig = (t: (key: string) => string) => { | ||
| const variant = posthog.getFeatureFlag("create_question_button_type"); | ||
|
|
||
| // Default to control variant (A) if no flag is set |
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.
You don't need a default here too, given you have a default branch in the switch case.
|
@claude fix formatting and check my inline review comments |
|
Claude finished @elisescu's task —— View job Completed ✓I've addressed both review comments:
Changes Madefront_end/messages/en.json
front_end/src/app/(main)/components/headers/hooks/useNavbarLinks.tsx
The changes have been committed and pushed to the PR branch. Job Run • Branch: |
|
@claude, the code doesn't build and formatting is still broken. |
|
Claude finished @elisescu's task —— View job Completed ✓I've fixed the formatting issue in the PR. Issues Found & Fixed
Changes Madefront_end/src/app/(main)/components/headers/hooks/useNavbarLinks.tsx
The formatting issue has been fixed and pushed to the PR branch. The code should now build successfully. Job Run • Branch: |
- Add PostHog feature flag integration for 'create_question_button_type' - Support 4 experiment variants (A, B, C, D) with different text and emphasis - Variant A: '+ Create' (no emphasis, control) - Variant B: '+ Create' (with emphasis) - Variant C: '+ Submit a Question' (no emphasis) - Variant D: '+ Submit a Question' (with emphasis) - Add dark/light theme support for emphasized buttons - Add translation key 'submitAQuestion' to en.json Closes #3364 Co-authored-by: Elis Popescu <[email protected]>
- Capitalize 'Submit a Question' in en.json - Remove unnecessary default value in useNavbarLinks.tsx Co-authored-by: Elis Popescu <[email protected]>
Co-authored-by: Elis Popescu <[email protected]>
24dd984 to
46c9bfe
Compare
Summary
Implements a 4-way A/B test for the question creation button in the top navigation bar. The implementation uses PostHog feature flags to dynamically render different button variants.
Changes
create_question_button_typeCloses #3364
Generated with Claude Code