refactor(frontier): remove AddOrganizationUsers RPC#479
Conversation
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThis pull request removes the Suggested reviewers
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
The latest Buf updates on your PR. Results from workflow Validate / validate (pull_request).
|
This RPC has been superseded by AddOrganizationMembers on the AdminService, which supports explicit role assignment per member. The server-side handler was already removed in raystack/frontier#1548. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
6558217 to
2adbc20
Compare
Summary
AddOrganizationUsersRPC, request message, and response message fromFrontierServiceAddOrganizationMembersonAdminService, which supports explicit role assignment per memberContext
AddOrganizationUsersadded members with a hardcoded viewer role and no way to specify the desired role. The replacementAddOrganizationMembers(onAdminService) accepts a role per member and is gated behind superadmin auth.Test plan
buf lintpassesbuf breakingpasses (WIRE category allows RPC removal)🤖 Generated with Claude Code