Skip to content

Commit

Permalink
Introduce fixes and more rigorous tests for 'Show on a map' feature (#…
Browse files Browse the repository at this point in the history
…26803)

This change introduces a fix for the PR #26214 to set the UserLocationMapURL correctly.
  • Loading branch information
n0toose authored Sep 29, 2023
1 parent c3b7231 commit 65e09a3
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 1 deletion.
1 change: 1 addition & 0 deletions routers/web/shared/user/header.go
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,7 @@ func PrepareContextForProfileBigAvatar(ctx *context.Context) {

ctx.Data["IsFollowing"] = ctx.Doer != nil && user_model.IsFollowing(ctx, ctx.Doer.ID, ctx.ContextUser.ID)
ctx.Data["ShowUserEmail"] = setting.UI.ShowUserEmail && ctx.ContextUser.Email != "" && ctx.IsSigned && !ctx.ContextUser.KeepEmailPrivate
ctx.Data["UserLocationMapURL"] = setting.Service.UserLocationMapURL

// Show OpenID URIs
openIDs, err := user_model.GetUserOpenIDs(ctx, ctx.ContextUser.ID)
Expand Down
1 change: 0 additions & 1 deletion routers/web/user/profile.go
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,6 @@ func userProfile(ctx *context.Context) {

ctx.Data["Title"] = ctx.ContextUser.DisplayName()
ctx.Data["PageIsUserProfile"] = true
ctx.Data["UserLocationMapURL"] = setting.Service.UserLocationMapURL

// prepare heatmap data
if setting.Service.EnableUserHeatmap {
Expand Down

0 comments on commit 65e09a3

Please sign in to comment.