-
Notifications
You must be signed in to change notification settings - Fork 14
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
Adjustments according to new no data page layout #456
Conversation
2b3c57d
to
fd5b6fa
Compare
fd5b6fa
to
04f8462
Compare
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.
Looks good 👍
The only issue I noticed during my testing is that the Cloud specific version of the GA no-data tab has some potential CSS/layout regressions. We likely won't know for sure until we have a Cloud branch with the core no-data changes.
@snake14 I've increased the required Matomo version to 5.0.0-rc5 (which will be released soon) and updated the test files. I will merge it, would be good if you could release a new version, so the plugin is compatible with last rc. |
Thank you @sgiehl . Would it be better to release now or wait till RC5 is released? I can see arguments for both ways. If we release ahead of time, customers might upgrade the plugin before upgrading core. If we release after RC5 is released, then there might be a gap between them upgrading core and the new plugin release becoming available. Of course, there aren't many customers using the RC releases, so it probably doesn't matter much. |
They shouldn't be able to update until rc5 was released. So should be safe to release already. |
Description:
refs matomo-org/matomo#21247
Review