Feature/cachaca#7
Conversation
|
|
||
| route.post('/webhooks', WebhookController.webhook) | ||
|
|
||
| route.post('/cachaca',CachacaController.index) |
There was a problem hiding this comment.
Se a função dessa rota é trazer a lista atual (pelo ano) de moradores o certo é ser um GET mesmo. Você pode usar query params para receber o ano. Ficaria algo do tipo route.get('/cachaca',CachacaController.index). E usaria .../cachaca?ano=2023
| const CachacaController = { | ||
| async index(req: Request, res: Response) { | ||
| try { | ||
| const body: ChachacaDTO = AnoSchema.parse(req.body) |
There was a problem hiding this comment.
Mudar para receber de req.query
| } | ||
| } | ||
|
|
||
| const AnoSchema = z.object({ |
There was a problem hiding this comment.
Aqui é só uma bilbioteca pra validar a entrada do usuário. Assim a gente traz erros melhores quando for obrigatório algum tipo de dado. Aqui acho que vai precisar só tirar o .object e deixar o number
|
|
||
| const cachacaService = { | ||
|
|
||
| async addMoradorAoRank(ano: number) { |
There was a problem hiding this comment.
Esse método faz coisa de mais
| 0, | ||
| ano, | ||
| ) | ||
| await CachacaRepository.addMoradorAoRankCachaca(toInsert) |
There was a problem hiding this comment.
só funciona na primeira vez, depois (com um ano diferente) vai tentar inserir o mesmo Id e vai dar erro
| 0, | ||
| ano, | ||
| ) | ||
| //verifica se o calouro tomou todas as cachaca para se tornar veterano se nao so na proxima vez que criar |
There was a problem hiding this comment.
Deveria estar no método que atualiza o morador, e se estiver tentanto atualizar ele pra veterano fazer essa verificação
3 participants
No description provided.