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

People: update the remove follower warning message #2673

Merged
merged 1 commit into from
Jan 22, 2016

Conversation

designsimply
Copy link
Contributor

Closes #2672

This pull request seeks to clarify the warning notice that appears after you remove a follower more clear so that it doesn't give site owners a false sense of privacy.

To test:

  1. Go to https://wordpress.com/people/followers and select a blog if prompted
  2. Click "Remove" next to one of the followers
  3. Check that the wording in the message has been updated to say:
If you remove this follower, he or she will no longer receive notifications about this site unless they re-follow.

screen shot 2016-01-21 at thu jan 21 3 14 53 pm

Closes #2672

This pull request seeks to clarify the warning notice that appears after you remove a follower more clear so that it doesn't give site owners a false sense of privacy.

To test: 

1. Go to https://wordpress.com/people/followers and select a blog if prompted
1. Click "Remove" next to one of the followers
1. Check that the wording in the message has been updated to say:

```
If you remove this follower, he or she will no longer receive notifications about this site unless they re-follow.
```
@designsimply designsimply added [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. People Management labels Jan 21, 2016
@alisterscott
Copy link
Contributor

I tested this change locally, and in English it looks good to me 👍
❓ I also tested another locale (Dutch) and found the new string isn't translated. Is this something that needs to be organized with Team Global before this is merged?

Dutch Before:
nl before

Dutch After:
nl after

English Before:
before

English After
after

@designsimply
Copy link
Contributor Author

Translations for updated strings are done through GlotPress and will typically happen at their own pace after an update like this.

@lancewillett
Copy link
Contributor

👍

Tested with Firefox 43.0.4 on OS X 10.11.2 on http://calypso.localhost:3000/people/followers/example.wordpress.com

Wording appears as expected.

@lancewillett
Copy link
Contributor

:shipit:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants