Skip to content

Conversation

@RianFonseca
Copy link

@RianFonseca RianFonseca commented Jan 13, 2025

Descrição das Mudanças

Descrição curta das mudanças

Instruções de QA

Passos para reproduzir:

Evidências (Se aplicável)

Checklist (Questionário de PR)

  • Todos os critérios de aceite da tarefa foram concluídos
  • Pode ser mergeado sem precisar de qualquer outra mudança
  • Implementação feita de somente uma história
  • Implementação esta presente no Swagger

Essa PR NÃO deve ser mergeada!

Evitamos de mergear, pois essa PR é apenas para estudo!

import { IsOptional } from 'class-validator';

export class FindByTextDto {
@IsOptional()
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

daria para adicionar uma outra validacao de @IsString()

Comment on lines +28 to +30
if (typeof result === 'string') {
return { message: result };
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Esse if tem que ficar dentro do caso de uso

Comment on lines +21 to +24
{
provide: TODO_REPOSITORY,
useClass: PrismaTodoRepository
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Esse cara vai ficar dentro do DatabaseModule e voce so vai precisar importar o DatabaseModule para o module

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