Skip to content
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

A/B test form design #1410

Merged
merged 5 commits into from Dec 20, 2018
Merged

A/B test form design #1410

merged 5 commits into from Dec 20, 2018

Conversation

gbirke
Copy link
Member

@gbirke gbirke commented Dec 18, 2018

Generate two different CSS files on build, one for each campaign
Make loaded CSS file dynamic
gbirke added a commit to wmde/fundraising-frontend-content that referenced this pull request Dec 19, 2018
Move link, identifier and label to data structure, use the data
structure in the template.

Make menu items translatable.

This is a preparation for being able to generate different menus as part
of an A/B test.
Move menu to ChoiceFactory
Add route name to the donation/new route. This route name follows the
pattern of the URL schema, described in ADR-005. We since we'd risk
breaking all banners, the URL stays the same for now.
Turn off form scrolling for group "B".
@gbirke gbirke changed the title [WIP] A/B test form design A/B test form design Dec 20, 2018
Copy link
Contributor

@tzhelyazkova tzhelyazkova left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The code look alright to me.
I user tested it as well and it covers the AC from the ticket.

@gbirke gbirke merged commit f999169 into master Dec 20, 2018
@gbirke gbirke deleted the test-from-design branch December 20, 2018 17:09
KaiNissen pushed a commit to wmde/fundraising-frontend-content that referenced this pull request Dec 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants