-
Notifications
You must be signed in to change notification settings - Fork 295
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
Enhancement/9935 features copy edit #10057
base: develop
Are you sure you want to change the base?
Conversation
Build files for 59efe60 are ready:
|
Size Change: +11 B (0%) Total Size: 1.98 MB
ℹ️ View Unchanged
|
JS tests are failing for non related reasons |
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.
Mostly looks good to me. Left two suggestions after which we can merge this PR.
'Reader Revenue Manager publication tracking (your Reader Revenue Manager account will still remain active)', | ||
'Users will no longer be able to sign in to your WordPress site using their Google Accounts.', | ||
'Users will not be able to create an account on your site using their Google Account (if account creation is enabled).', | ||
"Existing users who have only used Sign in With Google to sign in to your site will need to use WordPress' “Reset my password” to set a password for their account.", |
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.
"Existing users who have only used Sign in With Google to sign in to your site will need to use WordPress' “Reset my password” to set a password for their account.", | |
'Existing users who have only used Sign in With Google to sign in to your site will need to use WordPress’ “Reset my password” to set a password for their account.', |
'google-site-kit' | ||
), | ||
__( | ||
'Existing users who have only used Sign in with Google to sign in to your site will need to use WordPress\' "Reset my password" to set a password for their account', | ||
"Existing users who have only used Sign in With Google to sign in to your site will need to use WordPress' “Reset my password” to set a password for their account.", |
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.
"Existing users who have only used Sign in With Google to sign in to your site will need to use WordPress' “Reset my password” to set a password for their account.", | |
'Existing users who have only used Sign in With Google to sign in to your site will need to use WordPress’ “Reset my password” to set a password for their account.', |
Thanks @eugene-manuilov PR updated P.S Tests are failing for unrelated changes |
Summary
Addresses issue:
Relevant technical choices
PR Author Checklist
Do not alter or remove anything below. The following sections will be managed by moderators only.
Code Reviewer Checklist
Merge Reviewer Checklist