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 privacy policy/terms and conditions settings and display in bottom of certain pages. #13219
Conversation
Looks pretty good. I wonder if at some point we also might want to add "Powered by Matomo" there :) Off topic though :) |
$module = Common::getRequestVar('module', false); | ||
if ($module == 'Login') { | ||
return true; | ||
} |
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.
Actually that check is redundant, as all users on the login pages are anonymous
. But guess we could leave it there for a better understanding
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 see, will remove this if
* @link https://www.innocraft.com/ | ||
* @license For license details see https://www.innocraft.com/license | ||
*/ | ||
namespace Piwik\Plugins\PrivacyManager; |
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.
Can't use that license header here. Maybe we should rebase that branch, so it isn't anywhere in the history
9081bbd
to
38a13af
Compare
@sgiehl updated |
46fa2a4
to
05658b0
Compare
8e19743
to
e0adc69
Compare
…m of certain pages. (matomo-org#13219) * Add privacy policy/terms and conditions settings and display in bottom of certain pages. * tweak * simplify PrivacyManager::shouldRenderFooterLinks(). * Update system test files * removes typo * do not render view if no links available * Remove footer margin in embedded widget. * ensure footer margin doesn't change (for UI tests) * update ui files
Login page:
Dashboard (for anonymous user):
Another normal page (for anonymous user):
Widgetized (when enabled):
Fixes #12938