Repository navigation
Conversation
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
| @ApiProperty() | ||
| assoEmail: string; | ||
|
|
||
| @ApiProperty({ enum: ['CREATE', 'CLAIM'], nullable: true }) |
There was a problem hiding this comment.
Pk pas utiliser AssoAccessAction ici ?
| createdAt DateTime @default(now()) | ||
| userId String | ||
| assoEmail String @db.VarChar(100) | ||
| action String? @db.VarChar(10) |
There was a problem hiding this comment.
C'est soit CREATE soit CLAIM, right ?
| this.mailService.send( | ||
| email, | ||
| 'Validation d’accès association', | ||
| undefined, | ||
| { action, email }, | ||
| `Pour valider votre demande, utilisez ce lien : ${link}`, |
There was a problem hiding this comment.
En théorie, ce serait plus logique de mettre cette fonction dans le service (pareil en dessous)
| endAt: { gte: new Date() }, | ||
| OR: [ | ||
| { role: { isPresident: true } }, | ||
| { permissions: { some: { id: { in: ['manage_infos', 'manage_members', 'manage_roles', 'weekly'] } } } }, |
There was a problem hiding this comment.
Je mettrais peut être uniquement seulement la permission qui permet de changer les perms et les rôles des gens (je sais plus ce que c'est, probablement manage_members, puisque sinon on est locked out) mais en tout cas pas les weekly, ni manage_infos
| private isAllowedCasUserType(type?: UserType): boolean { | ||
| return type === UserType.STUDENT || type === UserType.TEACHER || type === UserType.EMPLOYEE; | ||
| } |
There was a problem hiding this comment.
Déplace ça au dessus de la tsdoc de la fonction du dessous
| import { MailService } from '../../../src/mail/mail.service'; | ||
|
|
||
| function tokenFromValidationEmail(mailService: MailService): string { | ||
| const calls = (mailService.send as jest.Mock).mock.calls; |
There was a problem hiding this comment.
quand t'auras rebase, faudra remplacer jest par vitest (et modifier les imports btw)
| type FakeUeWithOfs = FakeUe & { ueofs: FakeUeof[] }; | ||
| type FakeAssoMembers = { | ||
| role: FakeAssoMembershipRole; | ||
| users: { | ||
| user: FakeUser; | ||
| permissions: FakeAssoMembershipPermission[]; | ||
| }[]; | ||
| }[]; |
There was a problem hiding this comment.
En théorie, ces types vont dans fakedb.ts (sauf exceptions, mais tu me diras si c'est le cas)
| import ApplicationResDto from '../src/auth/application/dto/res/application-res.dto'; | ||
| import PermissionsResDto from '../src/auth/permissions/dto/res/permissions.dto'; | ||
|
|
||
| type JsonLikeVariant<T> = Partial<{ |
There was a problem hiding this comment.
T'as besoin de le mettre aussi dans le .ts ?
| MEDIA_DETACHED_LIFESPAN=1 | ||
|
|
||
| CAS_URL=https://cas.utt.fr/cas | ||
| CAS_URL=https://auth.utt.fr/cas |
There was a problem hiding this comment.
Même modif à faire dans .env.test.dist vu qu'elle est aussi set là bas
There was a problem hiding this comment.
J'avoue j'ai pas encore review ce fichier mais j'envoie quand même, le reste des commentaires entre temps
|
@tuturd pour info, en fixant les tests qui failaient à cause du merge, j'ai changé pour faire en sorte que par défaut un utilisateur est de type |
No description provided.