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

Implementar outros campos no PATCH para Agentes #205

Open
2 of 3 tasks
alessandrofeitoza opened this issue May 15, 2024 · 1 comment · May be fixed by #366
Open
2 of 3 tasks

Implementar outros campos no PATCH para Agentes #205

alessandrofeitoza opened this issue May 15, 2024 · 1 comment · May be fixed by #366
Assignees
Milestone

Comments

@alessandrofeitoza
Copy link
Member

alessandrofeitoza commented May 15, 2024

WHAT

Atualmente já temos uma request de PATCH para atualizar um agente, porém ela não funciona para todos os campos existentes.

WHY

Porque precisávamos entregar a funcionalidade de PATCH, e nesse sentido priorizamos alguns campos que fazem mais sentido de serem triviais.

ADDITIONAL INFO

Campos que não estão sendo atualizados:

  • CPF
  • CNPJ
  • E-mail pessoal
  • Telefone privado 1 com DDD
  • Telefone privado 2 com DDD
  • Complemento ou ponto de referência
  • Data de nascimento
  • Gênero
  • Orientação sexual
  • Raça/Cor
  • Agente itinerante
  • Tipo da conta bancária
  • Dados bancários
  • Comunidades tradicionais

Campos que requerem validações específicas:

  • Site
  • CPF
  • CNPJ
  • E-mail pessoal
  • Telefone público
  • E-mail público
  • Telefone privado 1 com DDD
  • Telefone privado 2 com DDD
  • CEP

Eventos incomuns:

O campo publicLocation não atualiza no navegador, mas é atualizado no endpoint, que por sua vez atualiza no navegador.

Não foram testados

  • Os campos de enviar imagens e arquivos.

Originally posted by @alessandrofeitoza in #193 (comment)

CHECKLIST

  • Criar novos (ou modificar) cenários os testes
  • Ajustar as fixtures
  • Ajustar a documentação (se necessário)
@alessandrofeitoza alessandrofeitoza added this to the Sprint 7 milestone May 15, 2024
@alessandrofeitoza
Copy link
Member Author

Essa issue depende da #230

@Akellymourab Akellymourab self-assigned this Jul 23, 2024
@Akellymourab Akellymourab linked a pull request Jul 24, 2024 that will close this issue
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 a pull request may close this issue.

2 participants