-
Notifications
You must be signed in to change notification settings - Fork 802
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
Add a "Support" link to the Plugin action links #2071
Conversation
Add a new link called "Support" to the /plugins page which will link to the in-dashboard Debugger and allow users to troubleshoot their site and contact Jetpack Support.
I don't think the debugger should be a full menu item. |
I do wonder, though, if it would be worth directly linking them to jetpack.me/contact-support -- perhaps with blog_id prefilled. |
We often have to ask the users to go through the process to send us the report from this debugger anyway, so it would be nice to have an easy way for them to generate and send this along with their question first, rather than having to ask for it in a reply. |
@@ -2728,6 +2728,7 @@ function plugin_action_links( $actions ) { | |||
return array_merge( | |||
$jetpack_home, | |||
array( 'settings' => sprintf( '<a href="%s">%s</a>', Jetpack::admin_url( 'page=jetpack_modules' ), __( 'Settings', 'jetpack' ) ) ), | |||
array( 'settings' => sprintf( '<a href="%s">%s</a>', Jetpack::admin_url( 'page=jetpack-debugger '), __( 'Support', 'jetpack' ) ) ), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this be array( 'support' => sprintf( '<a href="%s">%s</a>', Jetpack::admin_url( 'page=jetpack-debugger '), __( 'Support', 'jetpack' ) ) ),
instead, or am I crazy?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You are probably right, but I'm not sure.
@georgestephanis we wouldn't be adding a full menu item, just this link here: |
changed "settings" to "support". props @chase
I don't think support is the right label for this, if it's just linking to the Debug screen. Maybe just calling it Debug? Unsure. |
This makes sense, as there are plenty of opportunities for them to nav to support pages/forms from that page. Merging |
Add a "Support" link to the Plugin action links
Add a new link called "Support" to the /plugins page which will link to the in-dashboard Debugger and allow users to troubleshoot their site and contact Jetpack Support.