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

cpfOrCnpj #15

Open
mariohmol opened this issue Nov 9, 2019 · 12 comments
Open

cpfOrCnpj #15

mariohmol opened this issue Nov 9, 2019 · 12 comments
Labels
enhancement New feature or request

Comments

@mariohmol
Copy link
Owner

Field where the user can fill in a CPF or a CNPJ

@mariohmol mariohmol added the enhancement New feature or request label Nov 9, 2019
This was referenced Nov 11, 2019
@sirbaconjr
Copy link

Acho que isso dá para resumir em:

if (value.length <= 11) {
   return validateCPF(value);
}

return validateCNPJ(value);

Basicamente, valida como CPF até o tamanho extrapolar.

@mariohmol
Copy link
Owner Author

o lance desse é que alem do validar tem q fazer algo pra mascara ser maleável tb.. se consegui fazer um PR desse validate e alguns test cases ia ser massa!

@sirbaconjr
Copy link

VOu fazer um clone do seu repositório e fazer uns testes. Já já dou uma resposta.

@sirbaconjr
Copy link

Consegui, fiz Faker, Validator e Mask.

Pode me guiar em como fazer para gerar os compilados novos? Só fazer npm:dist? Ou vc cuida dessa parte?

@mariohmol
Copy link
Owner Author

seguinte.. cria um fork desse projeto.. aplica neles.. comita.. e compartilha aq.. a ideia é q se vc ir no Pull Requests.. oc consegue criar um PR enviando essas mudanças pra ca.. se precisar de ajuda podemos fazer um call q te mostro

@mariohmol
Copy link
Owner Author

foi mal.. agora q vi q ta sem os docs.. vou fazer aq

@sirbaconjr
Copy link

Blz então. Vou esperar você fazer, Já vou fazer o fork.

@mariohmol
Copy link
Owner Author

subi os docs
aproveitei pra ja deixar formatado todos os arquivos.. se puder atualizar sua branch

@mariohmol
Copy link
Owner Author

@mariohmol
Copy link
Owner Author

se tiver ruim ai pra atualziar a branch pode manda da forma como for q mergeio aqui =)

@sirbaconjr
Copy link

Okay, vou ver se faço essa parte hoje mais tarde. Eu faço o merge daqui e já abro o PR.

@BianorAraujo
Copy link

Vi que ainda estava aberto esse issue e adicionei a máscara para o value do cpfCnpj com os testes.
Criei o PR #81 para encerrar esse item.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants