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
Add campaign url builder widget #13546
Conversation
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.
Works as expected. Left some minor comments.
Btw. wouldn't that widget be a good place to suggest to install the MarketingCampaignsReporting
plugin, if it's not already installed? Guess we already do the same below the campaigns report?
$config->setCategoryId('Referrers_Referrers'); | ||
$config->setSubcategoryId('Referrers_URLCampaignBuilder'); | ||
$config->setName('Referrers_URLCampaignBuilder'); | ||
$config->setIsEnabled(SettingsPiwik::isSegmentationEnabled()); |
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.
Any reason for hiding the widget if segmentation is disabled?
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.
Nope, I'll update it.
@@ -0,0 +1,76 @@ | |||
/*! | |||
* Piwik - free/libre analytics platform |
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.
outdated header. but guess we may need to update all files at once one day...
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.
yeah at some point we need to update them all... will update it here anyway 👍
Pushed the update. We chose to not advertise the plugin there as we don't have installed on all of the accounts on the plugin for now and to keep things simple as we wouldn't want to show such a notice there etc. |
fix #12795