laravel-ab icon indicating copy to clipboard operation
laravel-ab copied to clipboard

Compatibility with Laravel 5

Open rafelsanso opened this issue 8 years ago • 15 comments

rafelsanso avatar Aug 03 '15 13:08 rafelsanso

There's a number of issues with this PR, like the console commands, tests. Also I don't think @jenssegers would like to remove support for L4. I've already got both L4 and L5 working in pr #7, it's just the tests that need updating (will try and update soon).

garygreen avatar Aug 03 '15 14:08 garygreen

I do prefer to keep support for both L4 and L5. But if it is too hard we could tag a new major release.

jenssegers avatar Aug 03 '15 15:08 jenssegers

@rafelsanso @garygreen I have been looking into the PR's last night. At first I wanted to keep supporting 4.x, 5.0 and 5.1. But after writing some code I kind of want to go for a new major release specifically for Laravel 5.1 LTS. I think keeping support for all 3 Laravel versions will make the codebase unnecessary complicated. I also have some new idea's that I want to integrate.

I can make separate branches with support for older versions, but they might not have all the features of the master branch.

jenssegers avatar Aug 04 '15 07:08 jenssegers

@jenssegers sounds good to me. You could use my PR as a base and strip out some L5 specific stuff (mainly the Helpers). I think using Config stuff inside Tester.php is bad practise, so it would be useful to bootstrap those in the service provider, as I have done in my PR. It makes tests easier as well.

garygreen avatar Aug 04 '15 07:08 garygreen

Guys, I want your opinion. Do you think it would be a good idea to integrate Google Analytics for stats tracking?

jenssegers avatar Aug 08 '15 08:08 jenssegers

Hi @jenssegers!

I think it's good idea. In fact, I use for statistical analytics. Style:

@if (AB::experiment('text1'))
        ga('send', 'event', 'experiments', 'homepage', 'text1');
@elseif (AB::experiment('text2'))
        ga('send', 'event', 'experiments', 'homepage', 'text2');
@endif

The advantage is we can obtain more data from Analytics visits, like the bounce rate and the number of page views.

And another thing that I see very interesting is save goals by javascript events. In my case I'm register the goal by pressing a button, not when you switch views.

rafelsanso avatar Aug 10 '15 13:08 rafelsanso

Makes sense to me. I assume it will be optional?

garygreen avatar Aug 10 '15 13:08 garygreen

I had a lot of issues integrating it with GA @rafelsanso. Can you show me a screenshot of what the reporting looks like with the ga('send', 'event', 'experiments', 'homepage', 'text1');. You can always send it to my email address if you do want to put it public :)

jenssegers avatar Aug 10 '15 13:08 jenssegers

Of course! Now I prepare an email with the example that I have. Can you tell me your email?

rafelsanso avatar Aug 10 '15 14:08 rafelsanso

You can send it to hello at jenssegers.be

jenssegers avatar Aug 10 '15 14:08 jenssegers

Replace the at with @ and remove the whitespaces :p

jenssegers avatar Aug 10 '15 14:08 jenssegers

Some progress of the project?

rafelsanso avatar Aug 25 '15 09:08 rafelsanso

@jenssegers @rafelsanso Any updates on this?

bart avatar Jul 17 '17 08:07 bart

Any updates on this ?

isbkch avatar Jan 16 '18 18:01 isbkch

I published a new package for ab testing that supports laravel 5.

ben182 avatar Feb 10 '19 11:02 ben182