-
Notifications
You must be signed in to change notification settings - Fork 18
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
feat: #44 - query mentorias #54
base: dev
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Peviii Abordagem do código em si ta correta, porem o certo seria uma query só, onde os filtros virão via variables do graphql, por exemplo "weeks: 1" ou "weeks: 2" e dai vc subtrai da data atual, no caso se o usuario não botar nada, apenas puxar todos os eventos, não precisa de currentDate. Outra coisa é adicionar a @ROLE de admin, somente usuários admin poderão acessar essa query
beleza então, vou começar a corrigir hoje estes pontos que você citou. |
consertei a função de filtrar eventos por semana e adicionei a Role de admin |
src/modules/event/event.service.ts
Outdated
...(weeks && { | ||
weeks, | ||
}), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no caso essa weeks não precisa estar aqui, você vai faz a conversão delas abaixo e só a conversão que precisa ir
src/modules/event/event.service.ts
Outdated
const subtractWeeks = currentDate.subtract(weeks, 'weeks'); | ||
|
||
const eventsPerWeek = await this.prisma.event.findMany({ | ||
where: { | ||
...options, | ||
...subtractWeeks.toDate(), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
aqui precisa fazer uma validação, só subtrair as semanas se weeks existir
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no caso seria um if() dentro do where mesmo né?
terminei a correção do código |
Criei 2 funções de listagem de mentorias no module de events, uma lista mentorias por semana e outra lista mentorias até o momento.