-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Update Gravatar to 3.0.0 #23701
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
Update Gravatar to 3.0.0 #23701
Conversation
Generated by 🚫 Danger |
|
| App Name | WordPress Alpha |
|
| Configuration | Release-Alpha | |
| Build Number | pr23701-b225a2e | |
| Version | 25.4.1 | |
| Bundle ID | org.wordpress.alpha | |
| Commit | b225a2e | |
| App Center Build | WPiOS - One-Offs #10873 |
|
| App Name | Jetpack Alpha |
|
| Configuration | Release-Alpha | |
| Build Number | pr23701-b225a2e | |
| Version | 25.4.1 | |
| Bundle ID | com.jetpack.alpha | |
| Commit | b225a2e | |
| App Center Build | jetpack-installable-builds #9914 |
|
About the dangermattic warning. I ran Edit: It turns out this is a false positive and the check will be removed. |
|
This is ready to review. |
etoledom
left a comment
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.
Looks good! 🎉


Fixes #
To test:
Me > My Profile > Update Profile Photo
Regression Notes
Avatars URLs. In fact, nothing has changed about Avatar URLs but let's just be extra careful for unexpected regressions.
Smoke test avatars throughout the app.
1 unit test added. 1 updated.
PR submission checklist:
RELEASE-NOTES.txtif necessary.Testing checklist: