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
Mention Piwik 2 is no longer supported (Piwik 2.18) #13448
Conversation
Changed it to at least view access. Would in theory need to show super user email login names or their email so they can ask them to update. |
{ | ||
if (!Piwik::isUserHasSomeViewAccess()) { | ||
return; | ||
} |
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.
Should this show for anonymous users?
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 would say so. I think we discussed it in the chat but not 100% sure. Maybe @mattab can confirm.
{% endif %} | ||
</div> | ||
</div> | ||
{% endblock %} |
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 guess this shouldn't be translated?
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.
No translation needed. Discussed it in the chat a while ago. It wouldn't be translated in time likely anyway and it'll be the last update.
@@ -49,6 +49,13 @@ public function getDefaultAction() | |||
return 'redirectToCoreHomeIndex'; | |||
} | |||
|
|||
public function piwikIsOutdated() | |||
{ | |||
Piwik::checkUserHasSomeViewAccess(); |
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.
Should this be shown to anonymous viewers?
Tried to keep it simple... showing warning to all admins/super users in reporting/admin UI:
And simple page showing more info
didn't add any of the piwik is now matomo image etc as didn't want to put it under GPL and I would say it is better to keep it straight to the point.
refs #11810