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

[3.6] Update opt-out usage tracking text. #23216

Merged
merged 2 commits into from
Apr 8, 2019
Merged

Conversation

timmyc
Copy link
Contributor

@timmyc timmyc commented Apr 5, 2019

All Submissions:

Changes proposed in this Pull Request:

This change seeks to make the opt-out explanation more clear. The suggested changes here were given by the Editorial team. /cc @seb86 could you let us know if this reads better to you?

Closes #23208

How to test the changes in this Pull Request:

  1. View the updated text in /wp-admin/admin.php?page=wc-settings&tab=account

Other information:

  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you written new tests for your changes, as applicable?
  • Have you successfully run tests with your changes locally?

Changelog entry

Fix: Make usage tracking opt-out instructions more clear.

@timmyc timmyc added this to the 3.6.0 milestone Apr 5, 2019
@timmyc timmyc requested a review from mikejolley April 5, 2019 20:03
@codecov
Copy link

codecov bot commented Apr 5, 2019

Codecov Report

Merging #23216 into master will increase coverage by 0.01%.
The diff coverage is 100%.

Impacted Files Coverage Δ Complexity Δ
...udes/admin/settings/class-wc-settings-accounts.php 94.07% <100%> (ø) 3 <0> (ø) ⬇️
...ncludes/tracks/events/class-wc-orders-tracking.php 0% <0%> (ø) 6% <0%> (+1%) ⬆️
includes/admin/class-wc-admin-setup-wizard.php 18.31% <0%> (+0.06%) 276% <0%> (-1%) ⬇️
includes/wc-template-functions.php 5.48% <0%> (+0.08%) 0% <0%> (ø) ⬇️
includes/class-wc-tax.php 80.94% <0%> (+0.26%) 131% <0%> (ø) ⬇️
includes/wc-core-functions.php 53.57% <0%> (+0.36%) 0% <0%> (ø) ⬇️

@timmyc timmyc changed the title Update opt-out usage tracking text. [3.6] Update opt-out usage tracking text. Apr 5, 2019
@helgatheviking
Copy link
Contributor

Minor change, but I would propose getting rid of "simply" so that it reads"

"To opt-out, leave this box unchecked. We won’t know this store exists and won’t collect any usage data. %s"

@willgorham
Copy link
Contributor

It might be worth clarifying specifically who the 'we' is. e.g. Woocommerce.com? Automattic?

@seb86
Copy link

seb86 commented Apr 7, 2019

I did mention WooCommerce.com in my rewrite when I posted the issue #23208. Words have power. Use them wisely please. This one mistake could impact WooCommerce. I hate to see that happen.

@@ -244,7 +244,7 @@ public function get_settings() {
'title' => __( 'Enable tracking', 'woocommerce' ),
'desc' => __( 'Allow usage of WooCommerce to be tracked', 'woocommerce' ),
/* Translators: %s URL to tracking info screen. */
'desc_tip' => sprintf( esc_html__( 'If you would rather opt-out, and do not check this box, we will not know this store exists and we will not collect any usage data. %s.', 'woocommerce' ), $tracking_info_text ),
'desc_tip' => sprintf( esc_html__( 'To opt-out, simply leave this box unchecked. We won’t know this store exists and won’t collect any usage data. %s.', 'woocommerce' ), $tracking_info_text ),
Copy link
Contributor

Choose a reason for hiding this comment

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

As suggested in https://wp.me/p2JRYi-402 by @californiakat, this one also alleviates the confusion around who is "we".

Suggested change
'desc_tip' => sprintf( esc_html__( 'To opt-out, simply leave this box unchecked. We won’t know this store exists and won’t collect any usage data. %s.', 'woocommerce' ), $tracking_info_text ),
'desc_tip' => sprintf( esc_html__( 'To opt out, leave this box unticked. Your store remains untracked, and no data will be collected. %s.', 'woocommerce' ), $tracking_info_text ),

@timmyc
Copy link
Contributor Author

timmyc commented Apr 8, 2019

Thanks everyone for the feedback on this text. Another round of edits based upon feedback from the PR here, and from some internal comments:

image

/cc @seb86 @willgorham @helgatheviking @psealock

@seb86
Copy link

seb86 commented Apr 8, 2019

@timmyc Much better

Copy link
Contributor

@psealock psealock left a comment

Choose a reason for hiding this comment

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

Thanks everyone, LGTM!

@timmyc timmyc merged commit 616af5c into master Apr 8, 2019
@californiakat
Copy link

In my original edit, opt out has no hyphen and that is correct. I didn't commit an error.

opt-out is an adjective or noun, opt out is a verb.

@claudiosanches claudiosanches deleted the update/tracks-opt-out branch August 2, 2019 21:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants