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

Show alert if guest author profile has thumbnail and is missing an email address #108

Open
danielbachhuber opened this issue Feb 18, 2013 · 4 comments

Comments

@danielbachhuber
Copy link
Contributor

If a guest author profile has a thumbnail, but is missing an email address, we should show a warning / require that an email address be entered.

Otherwise, the user will experience unexpected results on the frontend.

@danielbachhuber
Copy link
Contributor Author

... or we could generate a fake email address for the guest author so our filter hack works...

@danielbachhuber danielbachhuber modified the milestones: next, 3.1 Mar 12, 2014
@BrookeDot
Copy link

I am marking this as a bug because one of the unexpected front end results is trying to use a blank gravatar instead of showing the featured image/thumbnail.

IMO: We shouldn't require an email address we should just default to using the featured image. I'm not sure what the email address is used for though. If it's important than we should show an alert like was originally suggested.

If we are going to require an email I would suggest going with the filter method as it's better for the users than an alert.

cc @joshbetz

@danielbachhuber
Copy link
Contributor Author

I'm not sure what the email address is used for though.

Because get_avatar() also supports supplying a $user_id. However, should you have a user with ID 5 and a guest author with ID 5 (because guest authors are stored as posts), then the guest author would collide with the user.

@philipjohn philipjohn removed this from the next milestone Dec 3, 2016
@ituk
Copy link

ituk commented Nov 3, 2018

is this still valid? I encountered this issue today when I couldn't use get_avatar() with ID instead of user_email for guest authors without email. just opened a new thread for that, please have a look here: #616

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

No branches or pull requests

5 participants