Commit eb970a49 by Agnès Toulet Committed by GitHub

Org API: enrich add user to org endpoints with user ID in the response (#27551)

parent 8759a912
...@@ -212,7 +212,7 @@ Authorization: Bearer eyJrIjoiT0tTcG1pUlY2RnVKZTFVaDFsNFZXdE9ZWmNrMkZYbk ...@@ -212,7 +212,7 @@ Authorization: Bearer eyJrIjoiT0tTcG1pUlY2RnVKZTFVaDFsNFZXdE9ZWmNrMkZYbk
HTTP/1.1 200 HTTP/1.1 200
Content-Type: application/json Content-Type: application/json
{"message":"User added to organization"} {"message":"User added to organization","userId":1}
``` ```
## Admin Organizations API ## Admin Organizations API
...@@ -463,7 +463,7 @@ Content-Type: application/json ...@@ -463,7 +463,7 @@ Content-Type: application/json
HTTP/1.1 200 HTTP/1.1 200
Content-Type: application/json Content-Type: application/json
{"message":"User added to organization"} {"message":"User added to organization", "userId": 1}
``` ```
### Update Users in Organization ### Update Users in Organization
......
...@@ -123,7 +123,10 @@ func inviteExistingUserToOrg(c *models.ReqContext, user *models.User, inviteDto ...@@ -123,7 +123,10 @@ func inviteExistingUserToOrg(c *models.ReqContext, user *models.User, inviteDto
} }
} }
return Success(fmt.Sprintf("Existing Grafana user %s added to org %s", user.NameOrFallback(), c.OrgName)) return JSON(200, util.DynMap{
"message": fmt.Sprintf("Existing Grafana user %s added to org %s", user.NameOrFallback(), c.OrgName),
"userId": user.Id,
})
} }
func RevokeInvite(c *models.ReqContext) Response { func RevokeInvite(c *models.ReqContext) Response {
......
...@@ -4,6 +4,7 @@ import ( ...@@ -4,6 +4,7 @@ import (
"github.com/grafana/grafana/pkg/api/dtos" "github.com/grafana/grafana/pkg/api/dtos"
"github.com/grafana/grafana/pkg/bus" "github.com/grafana/grafana/pkg/bus"
"github.com/grafana/grafana/pkg/models" "github.com/grafana/grafana/pkg/models"
"github.com/grafana/grafana/pkg/util"
) )
// POST /api/org/users // POST /api/org/users
...@@ -35,12 +36,18 @@ func addOrgUserHelper(cmd models.AddOrgUserCommand) Response { ...@@ -35,12 +36,18 @@ func addOrgUserHelper(cmd models.AddOrgUserCommand) Response {
if err := bus.Dispatch(&cmd); err != nil { if err := bus.Dispatch(&cmd); err != nil {
if err == models.ErrOrgUserAlreadyAdded { if err == models.ErrOrgUserAlreadyAdded {
return Error(409, "User is already member of this organization", nil) return JSON(409, util.DynMap{
"message": "User is already member of this organization",
"userId": cmd.UserId,
})
} }
return Error(500, "Could not add user to organization", err) return Error(500, "Could not add user to organization", err)
} }
return Success("User added to organization") return JSON(200, util.DynMap{
"message": "User added to organization",
"userId": cmd.UserId,
})
} }
// GET /api/org/users // GET /api/org/users
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment