-
Notifications
You must be signed in to change notification settings - Fork 34
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
PLANET-7646 Add placeholder image to Author block #2489
Conversation
4d46380
to
cbbb5ab
Compare
/unhold 06e67f05-a64b-4adb-9dd9-d3837d84a50a
Test instance is ready 🚀🌑 tavros | admin | blocks report | CircleCI | composer-local.json ⌚ 2025.01.13 13:54:35 |
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 to me 👏
If author image is not available show MistryPerson(mm) avatar image
cbbb5ab
to
b59a2d2
Compare
/unhold fa87290a-eb3d-41f8-9d7b-8153addccefa
/unhold 7d0f9410-50a6-46b1-af2c-581a82db3b32
// Default avatar. | ||
if (defined('WP_DEBUG') && WP_DEBUG === true) { | ||
// Gravatar throws an error for local dev default_avatar URL, so the default value is used. | ||
$context['default_avatar'] = 'mm'; //Mystery Man |
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.
Thank you for commenting that it stands for "Mystery Man", I wouldn't have guessed 😂
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 to me 👏
Summary
if the author image is not available show MistryPerson(mm) avatar image
Ref: PLANET-7646
Testing
Please check the author image at the bottom of the post -
https://www-dev.greenpeace.org/test-tavros/story/1383/test-author-profile-image/
Same issue also fixed for author page