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

feat: #44 - query mentorias #54

Open
wants to merge 6 commits into
base: dev
Choose a base branch
from

Conversation

Peviii
Copy link
Contributor

@Peviii Peviii commented Jun 28, 2023

Criei 2 funções de listagem de mentorias no module de events, uma lista mentorias por semana e outra lista mentorias até o momento.

Copy link
Contributor

@oliveirabalsa oliveirabalsa left a 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

@Peviii
Copy link
Contributor Author

Peviii commented Jul 3, 2023

beleza então, vou começar a corrigir hoje estes pontos que você citou.

@Peviii
Copy link
Contributor Author

Peviii commented Jul 5, 2023

consertei a função de filtrar eventos por semana e adicionei a Role de admin

Comment on lines 226 to 228
...(weeks && {
weeks,
}),
Copy link
Contributor

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

Comment on lines 231 to 236
const subtractWeeks = currentDate.subtract(weeks, 'weeks');

const eventsPerWeek = await this.prisma.event.findMany({
where: {
...options,
...subtractWeeks.toDate(),
Copy link
Contributor

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

Copy link
Contributor Author

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é?

@Peviii
Copy link
Contributor Author

Peviii commented Jul 10, 2023

terminei a correção do código

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.

3 participants