Skip to content

chore: cleanup clients/v1 proto formatting #103

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.

Sign up for GitHub

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

chore: cleanup clients/v1 proto formatting #103

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions clients/v1/clients.proto
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,7 @@ service UsersService {
rpc GetUser(GetUserRequest) returns (GetUserResponse) {
option (sams_required_scopes) = "profile";
};

// GetUsers returns the list of SAMS users matching the provided IDs.
//
// NOTE: It silently ignores any invalid user IDs, i.e. the length of the return
Expand All @@ -33,6 +34,7 @@ service UsersService {
rpc GetUsers(GetUsersRequest) returns (GetUsersResponse) {
option (sams_required_scopes) = "profile";
};

// CreateUser creates a new SAMS user with the given email.
// Required scopes: sams::user::write
//
Expand All @@ -47,6 +49,7 @@ service UsersService {
rpc GetUserRoles(GetUserRolesRequest) returns (GetUserRolesResponse) {
option (sams_required_scopes) = "sams::user.roles::read";
};

// GetUserMetadata retrieves metadata for a SAMS user.
//
// Required scopes: 'sams::user.metadata::read' or metadata-namespace-specific
Expand All @@ -55,6 +58,7 @@ service UsersService {
// Cannot use 'sams_required_scopes' due to metadata-namespace-specific scopes.
// See docstring for required scopes.
}

// UpdateUserMetadata updates the metadata of a SAMS user.
//
// Required scopes: 'sams::user.metadata::write' or metadata-namespace-specific
Expand Down
Loading