Status report page link in ES notice does not work

mikelittle commented 1 year ago

During testing, I noticed that there was a ElesticPress notice displayed in the network admin pages. The notice inlcudes a link to the "Status EReport page". However, clicking the link leads to an error page with a "Sorry, you are not allowed to access this page." message.

The link is https://altis17beta.altis.dev/wp-admin/network/admin.php?page=elasticpress-status-report#failed-queries

Steps to reproduce: 1.

Private Zenhub Image

Acceptance criteria:

[ ] ...

For Altis Team Use

Ready for Work Checklist

Is this ticket ready to be worked on? See the Play Book Definition of Ready

[ ] Is the title clear?
[ ] Is the description clear and detailed enough?
[ ] Are acceptance criteria listed?
[ ] Have any (potential) dependencies been identified?
[ ] Is an estimate or time box assigned?
[ ] Is a priority label assigned?
[ ] Is this ticket added to an epic? (Optional)

Completion Checklist

Is this ticket done? See the Play Book Definition of Ready

[ ] Has the acceptance criteria been met?
[ ] Is the documentation updated (including README)?
[ ] Does any code/documentation meet project standards?
[ ] Are automatic tests in place to verify the fix or new functionality?
[ ] Or are manual tests documented (at least on this ticket)?
[ ] Are any Playbook/Handbook pages updated?
[ ] Has a new module release (patch/minor) been created/scheduled?
[ ] Have the appropriate backport labels been added to the PR?

svandragt commented 5 months ago

Had a look as I sometimes see a message in the admin saying there were errors in the last 24 hours, but the link in the message take me to the status page. The status page gives an access denied issue. This is because the menu item isn't registered as this repo removes the network admin menu. The subpage belongs to this:

https://github.com/humanmade/altis-enhanced-search/blob/5c342e383ce96cffa9930bca7895cc1acf92364f/inc/namespace.php#L172

commenting that out makes the status page appear.

It's probably better to just remove the Features, Settings and Sync submenu options, and keep Status Report and Index Health. I'm not sure about the implications of enabling Query Log, I suspect you'd want to hide that.

Hope that helps. It would be good to be able to debug errors, even if it's just for network admins.

SvetZitrka commented 4 months ago

I have the same problem

humanmade / altis-enhanced-search