From b5defc2a4a9081da282eebec19fd4d845e8e3229 Mon Sep 17 00:00:00 2001 From: Jonathan Miller Date: Mon, 25 May 2026 20:56:44 -0500 Subject: [PATCH] fix: populate org home sidebar with members, teams, and public member status (#183) The org home page template expects .Members, .NumMembers, .Teams, .IsOrganizationMember, .IsOrganizationOwner, and .IsPublicMember but the handler only set OrgOverviewMembers and OrgOverviewTeams (different key names). The sidebar rendered empty because the template variables were undefined. Fixed by adding all required data bindings and capturing the membersIsPublic map (previously discarded) as a callable function. Closes #183 Co-Authored-By: Claude Opus 4.6 (1M context) --- routers/web/org/home.go | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/routers/web/org/home.go b/routers/web/org/home.go index 7d06763e7e..3ea7c72c65 100644 --- a/routers/web/org/home.go +++ b/routers/web/org/home.go @@ -93,7 +93,7 @@ func home(ctx *context.Context, viewRepositories bool) { ListOptions: db.ListOptions{Page: 1, PageSize: 25}, } - members, _, err := organization.FindOrgMembers(ctx, opts) + members, membersIsPublic, err := organization.FindOrgMembers(ctx, opts) if err != nil { ctx.ServerError("FindOrgMembers", err) return @@ -102,6 +102,14 @@ func home(ctx *context.Context, viewRepositories bool) { const orgOverviewTeamsLimit = 5 ctx.Data["OrgOverviewMembers"] = members ctx.Data["OrgOverviewTeams"] = ctx.Org.Teams[:min(len(ctx.Org.Teams), orgOverviewTeamsLimit)] + ctx.Data["Members"] = members + ctx.Data["NumMembers"] = len(members) + ctx.Data["Teams"] = ctx.Org.Teams + ctx.Data["IsOrganizationMember"] = ctx.Org.IsMember + ctx.Data["IsOrganizationOwner"] = ctx.Org.IsOwner + ctx.Data["IsPublicMember"] = func(uid int64) bool { + return membersIsPublic[uid] + } ctx.Data["DisableNewPullMirrors"] = setting.Mirror.DisableNewPull ctx.Data["ShowMemberAndTeamTab"] = ctx.Org.IsMember || len(members) > 0 -- 2.52.0