Conversation
There was a problem hiding this comment.
I think you didn't accept the incoming changes when you merged main -> your branch, make sure to use what main has for these
There was a problem hiding this comment.
sorry, look at the lost comment to see what im talking about
There was a problem hiding this comment.
Request payload is swapped but should be fine:
{
"user_ids": ["uuid1", "uuid2"],
"role": "admin"
}
The response payload should be:
{
"data": {
"updated": [
{ "id": "uuid1", "email": "alice@utdallas.edu", "role": "admin" },
{ "id": "uuid2", "email": "bob@utdallas.edu", "role": "admin" }
]
}
}
you currently have:
{
"users": [
{
"auth_method": "passwordless",
"created_at": "string",
"email": "string",
"id": "string",
"profile_picture_url": "string",
"role": "hacker",
"supertokens_user_id": "string",
"updated_at": "string"
}
]
}
| } | ||
| if len(allowedOrigins) > 0 { | ||
| r.Use(cors.Handler(cors.Options{ | ||
| AllowedOrigins: allowedOrigins, |
There was a problem hiding this comment.
There is like a 95% chance changing the allowedOrigins is not allowed, you shouldn't have to edit configs
Check w/ Caleb
| } | ||
|
|
||
| if err := Validate.Struct(req); err != nil { | ||
| app.badRequestResponse(w, r, err) |
There was a problem hiding this comment.
give a proper error message, i thought you didn't implement this feature
| // @Summary Get or create application | ||
| // @Description Returns the authenticated user's hackathon application. If no application exists, creates a new draft application. | ||
| // @Tags hackers | ||
| // @Tags applications |
There was a problem hiding this comment.
I think you didn't accept the incoming changes when you merged main -> your branch, make sure to use what main has for these
2 participants
PR for super batch admin update