Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SELS-TASK-27] [BE]-Enable Authguards for the Edit A Category endpoint v.2 #32

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Fontilllas
Copy link
Collaborator

Asana
- link

[Commands]

  • docker compose up

[Pre-conditions]

  1. After running the command , you can now access the Postman and Dbeaver.
  2. Go to admin/login.
  3. You can now access the 'admin/category/:category_id/edit'

note: You can only access the endpoint as admin.

[Expected Output]

  • [x]Verify that only ADMIN can able to edit an existing category.

[Screenshots]

image

image

@Fontilllas Fontilllas self-assigned this Jan 9, 2024
@Fontilllas Fontilllas force-pushed the SPH-ELS-RUSSEL-27v2 branch 2 times, most recently from f5693fd to d1c7b97 Compare January 12, 2024 02:58
@Fontilllas Fontilllas changed the title [SELS-TASK] [BE]-Enable Authguards for the Edit A Category endpoint v.2 [SELS-TASK-27] [BE]-Enable Authguards for the Edit A Category endpoint v.2 Jan 19, 2024
@Fontilllas Fontilllas force-pushed the SPH-ELS-RUSSEL-27v2 branch 3 times, most recently from cd288fd to 6a49575 Compare January 19, 2024 10:52
Comment on lines 95 to 98
}

@UseGuards(AuthGuard)
@Put('admin/category/:category_id/edit')
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
}
@UseGuards(AuthGuard)
@Put('admin/category/:category_id/edit')
}
@UseGuards(AuthGuard)
@Put('admin/category/:category_id/edit')

Comment on lines 95 to 97
}

@UseGuards(AuthGuard)
Copy link
Collaborator

Choose a reason for hiding this comment

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

add newline

.gitignore Outdated
!.vscode/extensions.json

# Dbdata
.dbdata
Copy link
Collaborator

Choose a reason for hiding this comment

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

add newline

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.

2 participants