Add JWT authentication to PermissionController and update subOption titles

This commit is contained in:
faris Aljohari
2025-01-13 23:55:54 -06:00
parent 19cbc68fe0
commit ad4ec05d0a
2 changed files with 6 additions and 4 deletions

View File

@ -1,8 +1,9 @@
import { Controller, Get, Param } from '@nestjs/common'; import { Controller, Get, Param, UseGuards } from '@nestjs/common';
import { ApiTags, ApiOperation } from '@nestjs/swagger'; import { ApiTags, ApiOperation, ApiBearerAuth } from '@nestjs/swagger';
import { ControllerRoute } from '@app/common/constants/controller-route'; import { ControllerRoute } from '@app/common/constants/controller-route';
import { EnableDisableStatusEnum } from '@app/common/constants/days.enum'; import { EnableDisableStatusEnum } from '@app/common/constants/days.enum';
import { PermissionService } from '../services'; import { PermissionService } from '../services';
import { JwtAuthGuard } from '@app/common/guards/jwt.auth.guard';
@ApiTags('Permission Module') @ApiTags('Permission Module')
@Controller({ @Controller({
@ -11,7 +12,8 @@ import { PermissionService } from '../services';
}) })
export class PermissionController { export class PermissionController {
constructor(private readonly permissionService: PermissionService) {} constructor(private readonly permissionService: PermissionService) {}
@ApiBearerAuth()
@UseGuards(JwtAuthGuard)
@Get(':roleUuid') @Get(':roleUuid')
@ApiOperation({ @ApiOperation({
summary: ControllerRoute.PERMISSION.ACTIONS.GET_PERMISSION_BY_ROLE_SUMMARY, summary: ControllerRoute.PERMISSION.ACTIONS.GET_PERMISSION_BY_ROLE_SUMMARY,

View File

@ -37,7 +37,7 @@ export class PermissionService {
title, title,
subOptions: Object.entries(subOptions).map( subOptions: Object.entries(subOptions).map(
([subTitle, permissions]) => ({ ([subTitle, permissions]) => ({
title: subTitle, title: `MANAGE_${subTitle}`, // Prepend "MANAGE_" to subTitle
subOptions: permissions.map((permission) => ({ subOptions: permissions.map((permission) => ({
title: permission, title: permission,
isChecked: rolePermissions.includes(`${subTitle}_${permission}`), // Check if the role has the permission isChecked: rolePermissions.includes(`${subTitle}_${permission}`), // Check if the role has the permission