[13.0][ADD] website_ab_testing

[tarteo]

(for now) Simple A/B testing functionality.

A/B testing a conversion rate optimization tool.
In A/B testing you show multiple (mostly two) variants of the same page and determine
by the rate of conversion which version / variant is best.

First you want to configure a target:

1. Go to Website > A/B Testing > Targets;
2. click Create;
3. choose a name for your target e.g. 'More Sales' or 'More Members';
4. configure the triggers (triggers generate conversions e.g. when a certain page is visited or when a visitor bought a product from the webshop);
5. click Save.

Now we want to create different variants of our pages so we can potentially increase our conversion rate:

• Go to the website editor;
• go to your landing page;
• on the top right click on A/B Testing and New Variant to create a different version of the page;
• when you're done making variants make sure to enable A/B testing for the page by clicking the toggle in the A/B Testing menu

When your test has ran for some time we can find out which variant is best.
To find out which variant has lead to the most conversions, you can either
go to Website > A/B Testing > Conversions or click the Statistics button on the Target form.

[tarteo]

@OCA/website-maintainers It's impossible to comply with pre-commit flake8 and black:
https://github.com/OCA/website/pull/817/files#diff-607e05394c0dc3bcf4f71161020d011f8c070363191a5262f98770b318b793dfR8
flake8 wants a space between the comma and black don't.

[hbrunn]

@tarteo you can remove the comma altogether, this is a know bug in black

[bosd]

LGTM!

Created some eye candy for this one.

icon.pg

Alternative

[claudiagn]

functional review ok! LGTM 👍

[OCA-git-bot]

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

[dreispt]

/ocabot merge nobump

[OCA-git-bot]

What a great day to merge this nice PR. Let's do it!
Prepared branch 13.0-ocabot-merge-pr-817-by-dreispt-bump-nobump, awaiting test results.

[OCA-git-bot]

@dreispt your merge command was aborted due to failed check(s), which you can inspect on this commit of 13.0-ocabot-merge-pr-817-by-dreispt-bump-nobump.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

[dreispt]

Tests are failing https://app.travis-ci.com/github/OCA/website/jobs/500908529#L1010

[Tardo]

The problem is because the mock needs define the "website_id".

[gfcapalbo]

@tarteo You have a failing test : https://app.travis-ci.com/github/OCA/website/jobs/502825441#L1049
in

website/website_ab_testing/tests/test_ab_testing.py

Line 77 in 29ea93b

not_enabled = main.with_user(self.user_public).render()

your mock request has no website_id, it shoulkd be something similar to:
https://github.com/OCA/OCB/blob/13.0/addons/website_sale/tests/test_website_sale_pricelist.py#L197

[github-actions]

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

[bosd]

@tarteo What is the status of this one? Can this one be fixed?

[tarteo]

I'll look into it this week. The module works BTW it's just the tests failing.

[github-actions]

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

[bosd]

Please add no stale, would love to have this feature added!!

[dreispt]

Done.

[elvise]

@tarteo great PR!

What is the status ?

[elvise]

@tarteo just a gentle reminder 🙂

[tarteo]

@tarteo just a gentle reminder 🙂

Don't think I'm gonna work on this anymore (only if my employer requires it). The module works, only a test is failing because the mock is not complete. Feel free to make a PR on https://github.com/tarteo/website/tree/13-ab-testing to fix it, I'll merge right away.