Skip to content

Conversation

fxamauri
Copy link
Contributor

@fxamauri fxamauri commented Jul 1, 2024

Adiciona validação e formatação do novo CNPJ alfanumérico

Pontos de Atenção:

  • Tecnicamente, estamos introduzindo uma "breaking change" na validação do CNPJ, pois anteriormente não aceitávamos caracteres que não fossem números (havia testes validando isso que foram alterados nesta pr).
  • Agora, passamos a permitir caracteres alfanuméricos, o que quebra o comportamento anterior.
  • Precisamos considerar que essa mudança pode impactar nas versões anteriores a esta pr, que dependem da validação estritamente numérica.
  • Talvez seja interessante adicionar uma flag com option param nas funções alteradas para indicar que a validação deve considerar o novo formato alfanumérico.

Fico à disposição para pensarmos na melhor maneira de implementar isso.

resolve #361

@hyanmandian
Copy link
Member

Valeu demais @fxamauri ! Eu vou tentar arrumar um tempo essa semana pra fazer o review do teu PR e publica-lo. Muito obrigado!

Copy link
Member

@hyanmandian hyanmandian left a comment

Choose a reason for hiding this comment

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

Acredito que a change que eu sugeri referente ao options faca o restante do codigo mudar para poder suportar as duas versoes, dessa forma vou limitar meu review aqui ate que a gente resolva essa questao, tudo bem?

@fxamauri fxamauri force-pushed the feat/cnpj-alphanumeric branch 2 times, most recently from 2176021 to d610da3 Compare July 29, 2024 13:02
@fxamauri fxamauri force-pushed the feat/cnpj-alphanumeric branch from d610da3 to 29954e7 Compare July 29, 2024 13:03
@fxamauri fxamauri requested a review from hyanmandian July 29, 2024 13:05
@fxamauri fxamauri requested a review from hyanmandian July 30, 2024 17:02
@hyanmandian
Copy link
Member

@fxamauri estou trabalhando num refactoring do projeto, ja vou incluir sua mudanca nesse refactoring. Desculpa nao ter te dado retorno antes, mas agora estou voltando a manter o projeto!

@hyanmandian
Copy link
Member

Ja inclui no refactor!

#398

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.

Adicionar suporte para o novo CNPJ alfanumérico

2 participants