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

Remove System Info And Send People to WP's Site Health Instead #862

Closed
fpcorso opened this issue Jul 30, 2020 · 4 comments
Closed

Remove System Info And Send People to WP's Site Health Instead #862

fpcorso opened this issue Jul 30, 2020 · 4 comments

Comments

@fpcorso
Copy link
Contributor

fpcorso commented Jul 30, 2020

Right now, our system info is just a text dump into a textarea. In order for an admin to identify issues, they have to scroll through the textarea and know enough about WordPress and Popup Maker to determine if something is an issue.

We should update it to make it clear what is a problem and what isn't. Probably should also set up some documentation to help explain all the parts of the information too.

I like how some plugins call it "System Status" instead of "System Info". Maybe we consider renaming it?

@fpcorso fpcorso added this to the v1.12 milestone Jul 30, 2020
@fpcorso fpcorso modified the milestones: v1.12, v1.13 Aug 6, 2020
@fpcorso fpcorso modified the milestones: v1.13, v1.14 Sep 30, 2020
@fpcorso fpcorso modified the milestones: v1.14, v1.15 Nov 2, 2020
@fpcorso fpcorso modified the milestones: v1.15, v1.16 Dec 3, 2020
@fpcorso
Copy link
Contributor Author

fpcorso commented Jan 5, 2021

@danieliser Something I have seen a few other plugins do recently is to remove their "system info" pages and instead add some checks to the core "Site Health" page.

For example, Monster Insights, WP Rocket, and Yoast added these:
image

Then, to get system info, we just ask them to copy the system info as shown:
image

This way, we are hooking into something the admins are already checking and it is much more user-friendly to spot an issue as opposed to having to scroll through our current text.

Thoughts on going with this strategy as opposed to my original proposed improvements on this issue?

@danieliser
Copy link
Member

@fpcorso That second set of approaches looks good. Hook into existing stuff is way smarter.

@fpcorso
Copy link
Contributor Author

fpcorso commented Jan 8, 2021

@danieliser I reviewed our support info and what is in the Site Health Info that can be saved to clipboard and all of the info we have is already there so there's nothing for us to add.

Plus, all the things we check for are already in the Site Health checks too.

So, we could remove and send people there already. Should we proceed with this plan then? Would there be anything additional we want to add?

It's one less thing we have to maintain, other people are actively working on it, and it's something that shows up in the dashboard for most admins now. Plus, Site Health was added to WP in 5.2 and over 98% of our users are using 5.2 or newer so almost all already have access to it.

@danieliser
Copy link
Member

@fpcorso perfect.

@fpcorso fpcorso changed the title Improve System Info Remove System Info And Send People to WP's Site Health Instead Jan 12, 2021
fpcorso added a commit that referenced this issue Jan 12, 2021
fpcorso added a commit that referenced this issue Jan 12, 2021
fpcorso added a commit that referenced this issue Jan 12, 2021
@fpcorso fpcorso mentioned this issue Jan 12, 2021
12 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants