louis/targo-backend
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request 'fix(employees): fix profile route to actually include email in route as param' (#4) from auth-session-fullstack into main

Browse Source 
Reviewed-on: Targo/targo_backend#4
This commit is contained in:
Nicolas 2025-08-15 17:03:20 -04:00
commit bc3a3b96d8
2 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
docs/swagger/swagger-spec.json
View File

@ -384,7 +384,7 @@
]
}
},
"/employees/profile": {
"/employees/profile/{email}": {
"get": {
"operationId": "EmployeesController_findOneProfile",
"parameters": [
@ -392,6 +392,7 @@
"name": "email",
"required": true,
"in": "path",
"description": "Identifier of the employee",
"schema": {
"type": "string"
}

5
src/modules/employees/controllers/employees.controller.ts
View File

@ -51,11 +51,12 @@ export class EmployeesController {
return this.employeesService.findOne(id);
}
@Get('profile')
@Get('profile/:email')
@ApiOperation({summary: 'Find employee profile' })
@ApiParam({ name: 'email', type: String, description: 'Identifier of the employee' })
@ApiResponse({ status: 200, description: 'Employee profile found', type: EmployeeProfileItemDto })
@ApiResponse({ status: 400, description: 'Employee profile not found' })
findOneProfile(@Param('email', ParseIntPipe)email: string): Promise<EmployeeProfileItemDto> {
findOneProfile(@Param('email') email: string): Promise<EmployeeProfileItemDto> {
return this.employeesService.findOneProfile(email);
}