Skip to content

feat: add a paginated organization members endpoint #16835

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

Merged
merged 19 commits into from
Mar 10, 2025
Merged

feat: add a paginated organization members endpoint #16835

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
cea4d3c
feat: add paginated members route
brettkolodny Mar 6, 2025
e97c9db
feat: add request and response types
brettkolodny Mar 6, 2025
34003dc
feat: create new sql query and generate code
brettkolodny Mar 6, 2025
9edb088
fix: add dbauthz tests
brettkolodny Mar 6, 2025
ef40967
feat: implement endpoint
brettkolodny Mar 6, 2025
6aaddaf
fix: correct endpoint documentation
brettkolodny Mar 6, 2025
a724f48
fix: lint errors
brettkolodny Mar 6, 2025
4925592
fix: generated docs from new endpoint
brettkolodny Mar 6, 2025
902538c
fix: remove unneeded comments
brettkolodny Mar 6, 2025
06ff95c
wip
brettkolodny Mar 7, 2025
560305d
Merge branch 'main' into brett/org-members-pagination-backend
brettkolodny Mar 7, 2025
76a4cbf
fix: incorrectly showing type in panic
brettkolodny Mar 7, 2025
737ac9f
fix: update docs
brettkolodny Mar 7, 2025
8910df3
fix: change dbauthz test back to proper one
brettkolodny Mar 7, 2025
0e579a2
fix: un-nest route
brettkolodny Mar 7, 2025
73e356c
fix: add comment
brettkolodny Mar 10, 2025
24332f4
fix: update test to account for dbmem and check for returned member
brettkolodny Mar 10, 2025
7af0b60
fix: update test to check for accurate count
brettkolodny Mar 10, 2025
cdc0322
fix: make dbmem implementation match expected behaior around 0 limit
brettkolodny Mar 10, 2025
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
Prev Previous commit
Next Next commit
fix: lint errors
  • Loading branch information
@brettkolodny
brettkolodny committed Mar 6, 2025
commit a724f484c93e201e8d6ae95b5f9bf2b330971c4b
4 changes: 2 additions & 2 deletions coderd/database/dbmem/dbmem.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9584,7 +9584,7 @@ func (q *FakeQuerier) OrganizationMembers(_ context.Context, arg database.Organi
return tmp, nil
}

func (q *FakeQuerier) PaginatedOrganizationMembers(ctx context.Context, arg database.PaginatedOrganizationMembersParams) ([]database.PaginatedOrganizationMembersRow, error) {
func (q *FakeQuerier) PaginatedOrganizationMembers(_ context.Context, arg database.PaginatedOrganizationMembersParams) ([]database.PaginatedOrganizationMembersRow, error) {
err := validateDatabaseType(arg)
if err != nil {
return nil, err
Expand All @@ -9602,7 +9602,7 @@ func (q *FakeQuerier) PaginatedOrganizationMembers(ctx context.Context, arg data
}

if skippedMembers < int(arg.OffsetOpt) {
skippedMembers += 1
skippedMembers++
continue
}

Expand Down
5 changes: 4 additions & 1 deletion coderd/members.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -212,7 +212,7 @@ func (api *API) paginatedMembers(rw http.ResponseWriter, r *http.Request) {
return
}

memberRows := make([]database.OrganizationMembersRow, len(paginatedMemberRows))
memberRows := make([]database.OrganizationMembersRow, 0)
for _, pRow := range paginatedMemberRows {
row := database.OrganizationMembersRow{
OrganizationMember: pRow.OrganizationMember,
Expand All @@ -227,6 +227,9 @@ func (api *API) paginatedMembers(rw http.ResponseWriter, r *http.Request) {
}

members, err := convertOrganizationMembersWithUserData(ctx, api.Database, memberRows)
if err != nil {
httpapi.InternalServerError(rw, err)
}

resp := codersdk.PaginatedMembersResponse{
Members: members,
Expand Down
Loading