Skip to content
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

Merged
merged 6 commits into from
Sep 29, 2023
Merged

Conversation

sgiehl
Copy link
Member

@sgiehl sgiehl commented Sep 18, 2023

Description:

refs matomo-org/matomo#21247

Review

@sgiehl sgiehl marked this pull request as ready for review September 26, 2023 11:55
Copy link
Contributor

@snake14 snake14 left a 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.

@sgiehl
Copy link
Member Author

sgiehl commented Sep 29, 2023

@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.

@sgiehl sgiehl merged commit 04ef4b7 into 5.x-dev Sep 29, 2023
5 checks passed
@sgiehl sgiehl deleted the nodatalayout branch September 29, 2023 11:08
@snake14
Copy link
Contributor

snake14 commented Oct 1, 2023

@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.

@sgiehl
Copy link
Member Author

sgiehl commented Oct 1, 2023

They shouldn't be able to update until rc5 was released. So should be safe to release already.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants