Feature/1164#1165
Conversation
|
@DominikLudwiczak is attempting to deploy a commit to the Listinai Team on Vercel. A member of the Team first needs to authorize it. |
DominikLudwiczak
temporarily deployed
to
build-pr
GitHub Actions
Inactive
![sentry[bot]](https://avatars.githubusercontent.com/in/12637?s=60&v=4){kind=small}
| import { AddTeamMemberDto } from '@gitroom/nestjs-libraries/dtos/settings/add.team.member.dto'; | ||
| import { ApiTags } from '@nestjs/swagger'; | ||
| import { AuthorizationActions, Sections } from '@gitroom/backend/services/auth/permissions/permission.exception.class'; | ||
| import { User } from 'facebook-nodejs-business-sdk'; |
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
There was a problem hiding this comment.
right, I pushed change
DominikLudwiczak
temporarily deployed
to
build-pr
GitHub Actions
Inactive
| @Body() body: ChangePasswordDto | ||
| ) { | ||
| const userWithPassword = await this._userService.getUserById(user.id); | ||
| if(!userWithPassword || !AuthChecker.comparePassword(body.oldPassword, userWithPassword.password)) { |
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
There was a problem hiding this comment.
right, changed it and also in setting tab on frontend part I added condition that only LOCAL users can see change password tab
DominikLudwiczak
temporarily deployed
to
build-pr
GitHub Actions
Inactive
| const userWithPassword = await this._userService.getUserById(user.id); | ||
| if(userWithPassword.providerName !== 'LOCAL') { | ||
| return false; | ||
| } | ||
| if(!userWithPassword || !AuthChecker.comparePassword(body.oldPassword, userWithPassword.password)) { |
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
DominikLudwiczak
temporarily deployed
to
build-pr
GitHub Actions
Inactive
| @Controller('/auth') | ||
| export class AuthController { | ||
| constructor( | ||
| constructor( |
There was a problem hiding this comment.
Please don't modify any files unrelated to this PR.
| directory: "/" # Location of package manifests | ||
| schedule: | ||
| interval: "daily" | ||
| interval: "weekly" |
There was a problem hiding this comment.
Please don't modify any files unrelated to this PR.
|
This PR is stale because it has been open for 90 days with no activity. |
2 participants
What kind of change does this PR introduce?
applies password change feature
Why was this change needed?
#1164
Other information:
Checklist:
Put a "X" in the boxes below to indicate you have followed the checklist;