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
Process goal conversions for device data (type, brand, model) #10026
Conversation
8fdaf48
to
0897851
Compare
0897851
to
3fb88b3
Compare
* @param Action|null $action | ||
* @return mixed | ||
*/ | ||
public function onAnyGoalConversion(Request $request, Visitor $visitor, $action) |
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.
@mattab this will add columns to log_conversion
. So for the other dimensions. I presume it's maybe not as much of a major update to add these columns or shall we merge into Piwik 3?
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.
True, there will three columns be added to log_conversion
table. Can also rebase and recreate the pull request on 3.x
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.
I think it would be better to recreate PR against 3.x because of the schema change. Otherwise looks 👍
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.
I'll reissue another PR for 3.x-dev
branch so we can decide where to merge
3fb88b3
to
cce63e0
Compare
cce63e0
to
2e73c0b
Compare
Can be closed if we decide to merge that in 3.x only. |
Let's close this one. The actual issue is in mid term milestone anyway and ideally we issue all PRs against Piwik 3 first anyway. Especially when it's a new feature since Piwik 2 is in LTS |
fixes #9777