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

Add a "Support" link to the Plugin action links #2071

Merged
merged 2 commits into from
Jun 18, 2015

Conversation

mikeyarce
Copy link
Member

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.

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.
@mikeyarce mikeyarce added the [Type] Enhancement Changes to an existing feature — removing, adding, or changing parts of it label May 1, 2015
@jeherve jeherve added the General label May 1, 2015
@jeherve jeherve added this to the 3.6 milestone May 1, 2015
@georgestephanis
Copy link
Member

I don't think the debugger should be a full menu item.

@georgestephanis
Copy link
Member

I do wonder, though, if it would be worth directly linking them to jetpack.me/contact-support -- perhaps with blog_id prefilled.

@chaselivingston
Copy link
Contributor

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' ) ) ),
Copy link
Contributor

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?

Copy link
Member Author

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.

@mikeyarce
Copy link
Member Author

@georgestephanis we wouldn't be adding a full menu item, just this link here:
screen shot 2015-05-01 at 11 11 03 am

changed "settings" to "support".  props @chase
@dereksmart dereksmart added [Status] Needs Review To request a review from fellow Jetpack developers. Label will be renamed soon. and removed [Status] In Progress labels May 13, 2015
@georgestephanis
Copy link
Member

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.

@dereksmart
Copy link
Member

This makes sense, as there are plenty of opportunities for them to nav to support pages/forms from that page. Merging

dereksmart added a commit that referenced this pull request Jun 18, 2015
Add a "Support" link to the Plugin action links
@dereksmart dereksmart merged commit 1a68043 into Automattic:master Jun 18, 2015
@dereksmart dereksmart removed the [Status] Needs Review To request a review from fellow Jetpack developers. Label will be renamed soon. label Jun 18, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
General [Status] Requires String Changes [Type] Enhancement Changes to an existing feature — removing, adding, or changing parts of it
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants