Skip to content

Batch role updating route#48

Closed
tharunthunder wants to merge 1 commit intomainfrom
feat/superAdminBatchRoleUpdate
Closed

Batch role updating route#48
tharunthunder wants to merge 1 commit intomainfrom
feat/superAdminBatchRoleUpdate

Conversation

@tharunthunder
Copy link
Copy Markdown
Collaborator

Finished the issue

@tharunthunder tharunthunder requested a review from balebbae March 5, 2026 22:58
Comment thread cmd/api/middlewares.go
user, _ := ctx.Value(userContextKey).(*store.User)
return user
}
func (app *application) SuperAdminUpdate() func(http.Handler) http.Handler {
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nah bro what is this and what are these comments 😭

Copy link
Copy Markdown
Collaborator

@balebbae balebbae left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Make sure it passes the audits and read that issue again. You're implementing an endpoint.

@tharunthunder tharunthunder deleted the feat/superAdminBatchRoleUpdate branch March 26, 2026 14:29
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.

API: Batch update user roles (PATCH /v1/superadmin/users/role)

2 participants