-
Notifications
You must be signed in to change notification settings - Fork 645
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
Profile Picture on Profile Page #2657
Conversation
Hi @HBrock, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution! TTYL, DNFBOT; |
@HBrock, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR. |
@@ -6,9 +7,13 @@ | |||
|
|||
|
|||
@section SideColumn { | |||
@if (String.IsNullOrEmpty(Model.EmailAddress)) | |||
@* Change to not null or empty *@ |
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.
What is meant by this comment? Remove it if not needed.
Can you limit the PR to include just the |
Is there away I can limit it to just that one file for github? |
That will need some git-fu. Here's what you could do:
|
The changes should be fixed...If I did not do anything wrong. |
I seen I had an error with the last update. It should be working now. |
I change the logic for show the Gravatar in Profiles.cshtml.cs