-
-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
Fix invalid reference in feeds template #1820
Conversation
models/action.go
Outdated
@@ -142,6 +142,11 @@ func (a *Action) ShortActUserName() string { | |||
return base.EllipsisString(a.GetActUserName(), 20) | |||
} | |||
|
|||
func (a *Action) GetActAvatar() string { |
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.
Missing comment on this function
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.
Good catch, updated
maybe add an integration test for the dashboard so that any template render error could be catched next time. @ethantkoenig |
Otherwise LGTM |
LGTM |
@lunny Added an integration test |
bced2d3
to
f8a7065
Compare
Rebased to get update to Immediately after rebasing, I realized that it wasn't necessary, since a bad signature won't break a PR build. Sorry for any inconvenience. |
@ethantkoenig don't worry, I will merge this after CI completed. |
Hi Thanks I tested now and no problem on dashboard display |
Fixes the following error on the dashboard page: (oversight on my part from #1779)
see #1812 (comment)