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

PHP7.x warning, dismissable please #21558

Open
BloodyIron opened this issue Nov 16, 2023 · 2 comments
Open

PHP7.x warning, dismissable please #21558

BloodyIron opened this issue Nov 16, 2023 · 2 comments
Labels
Enhancement For new feature suggestions that enhance Matomo's capabilities or add a new report, new API etc.

Comments

@BloodyIron
Copy link

I'm on v4.15.1

My system running Matomo also runs another PHP tool on the same host that currently limits the running PHP version max to 7.4. I am of course using the Ondrej PPA (THANKS ONDREJ!!!) to continue to get PHP7.4 updates/fixes/etc as well as the Ondrej Apache2 PPA for that part too (as per Ondrej's documented recommendation).

Now I KNOW that PHP7.4 is not ideal and I want to move to 8.x, however the other tool is likely months away from having proper PHP8.x operability (work is already underway, this is explicitly confirmed).

So, what I reallllyyy would like is a way to dismiss the forced banner for PHP7.4 warning, and instead have that move to the system check line which mentions the PHP version. That line has a green checkbox, mentions the PHP version. IMO that line could mention the recommendation of moving to PHP8.whatever. As of right now the PHP7.4 warning cannot be dismissed (so far as I can tell), and it would really be appreciated if that changed.

Not exactly a high priority change, and I know this is not ideal, but such is the life of IT at times. If this were added it would be appreciated! Thanks :)

@BloodyIron BloodyIron added Enhancement For new feature suggestions that enhance Matomo's capabilities or add a new report, new API etc. To Triage An issue awaiting triage by a Matomo core team member labels Nov 16, 2023
@sgiehl
Copy link
Member

sgiehl commented Nov 17, 2023

Hi @BloodyIron,

The notification for outdated PHP versions being used is only visible for super users. So using a normal user or admin account won't show the notification.

As far as I know we don't plan to make that notification dismiss-able, as we want to ensure that super users are aware of it.
But I'll let the product team consider this.

@sgiehl sgiehl removed the To Triage An issue awaiting triage by a Matomo core team member label Nov 17, 2023
@sgiehl sgiehl added this to the For Prioritization milestone Nov 17, 2023
@BloodyIron
Copy link
Author

Hi @BloodyIron,

The notification for outdated PHP versions being used is only visible for super users. So using a normal user or admin account won't show the notification.

As far as I know we don't plan to make that notification dismiss-able, as we want to ensure that super users are aware of it. But I'll let the product team consider this.

Well my current scale is a user size of 1, so I really would rather have an actual emergency message be non-dismissable like that, and instead have a recommendation at the PHP version check line along the lines of "You really should be on PHP8.x, but the version you have still works". As it is now, it's not really warranted to be non-dismissable since the tool clearly still works, and as I've mentioned, I still get PHP7.4 updates (due to Ondrej being a real ma fugga).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement For new feature suggestions that enhance Matomo's capabilities or add a new report, new API etc.
Projects
None yet
Development

No branches or pull requests

2 participants