Skip to content

fix: include organization member roles#66

Merged
zhangyangyu merged 1 commit into
mainfrom
codex/organization-membership-fields
May 30, 2026
Merged

fix: include organization member roles#66
zhangyangyu merged 1 commit into
mainfrom
codex/organization-membership-fields

Conversation

@zhangyangyu
Copy link
Copy Markdown
Collaborator

Description

Adds membership role and state fields to organization member list responses so clients can render admin/member state without separate membership lookups. The service now returns active membership metadata with listed users while preserving the distinct invitation role vocabulary used by pending invitations.

Tracking

N/A

Tests

  • make check
  • go test ./internal/rest -run TestOrganizationMemberHandlers -v
  • go test ./internal/service

SSOT Drift Prevention Checklist

  • Checked that this PR does not violate boundaries in docs/architecture.md or docs/module-contracts.md
  • Package structure and dependencies are unchanged; no SSOT doc update needed
  • Test conventions and coverage expectations are unchanged; no docs/test-strategy.md update needed
  • CI, smoke, regression, and packaging workflows are unchanged; no docs/ci.md update needed

@zhangyangyu zhangyangyu merged commit fb55c18 into main May 30, 2026
12 checks passed
@zhangyangyu zhangyangyu deleted the codex/organization-membership-fields branch May 30, 2026 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant