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

Enhancement: Add more issues found to the Troubleshooting page [Innovation Day] #2153

Open
wants to merge 16 commits into
base: master
Choose a base branch
from

Conversation

andrasguseo
Copy link
Contributor

@andrasguseo andrasguseo commented Jul 19, 2024

🎫 Ticket

[Innovation Day]

🗒️ Description

On the Events/Tickets > Troubleshooting page there is a section displaying issues found on the website.
There are limited to 4 issues.
This PR adds another 3 issues that are common and can help prevent the customers opening support tickets.
The added issues are:

  • PHP version check: flags if the PHP version is less than 8.0
  • The default PHP time zone is NOT set to UTC, as this can cause issues with event times
  • A caching plugin is detected

Note, there's no changelog entry yet.

🎥 Artifacts

Screenshot of the Troubleshooting page showing the 3 new issues:
https://www.dropbox.com/scl/fi/5zkjoef4qg3fs0tt8pozy/shot_240719_205012.jpg?rlkey=zpx9c1j2ugdg5hghmlckspwp6&e=1&dl=0

✔️ Checklist

  • Changelog entry in the readme.txt file.
  • Code is covered by NEW wpunit or integration tests.
  • Code is covered by EXISTING wpunit or integration tests.
  • Are all the required tests passing?
  • Automated code review comments are addressed.
  • Have you added Artifacts?
  • Check the base branch for your PR.
  • Add your PR to the project board for the release.

@andrasguseo andrasguseo added the code review Status: requires a code review. label Jul 19, 2024
@andrasguseo andrasguseo self-assigned this Jul 19, 2024
Copy link
Member

@bordoni bordoni left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Address the code style issues pointed out by PHPCS.

@andrasguseo andrasguseo added the needs release Needs an associated release in Central before merging. label Jul 22, 2024
@andrasguseo andrasguseo requested a review from bordoni July 22, 2024 16:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code review Status: requires a code review. needs release Needs an associated release in Central before merging.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants