js-brasil icon indicating copy to clipboard operation
js-brasil copied to clipboard

cpfOrCnpj

Open mariohmol opened this issue 5 years ago • 11 comments

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

mariohmol avatar Nov 09 '19 01:11 mariohmol

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.

sirbaconjr avatar Dec 27 '19 01:12 sirbaconjr

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!

mariohmol avatar Dec 27 '19 01:12 mariohmol

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

sirbaconjr avatar Dec 27 '19 01:12 sirbaconjr

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?

sirbaconjr avatar Dec 27 '19 03:12 sirbaconjr

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 avatar Dec 27 '19 14:12 mariohmol

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

mariohmol avatar Dec 27 '19 14:12 mariohmol

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

sirbaconjr avatar Dec 27 '19 14:12 sirbaconjr

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

mariohmol avatar Dec 27 '19 15:12 mariohmol

https://github.com/mariohmol/js-brasil/blob/master/CONTRIBUTING.md

mariohmol avatar Dec 27 '19 15:12 mariohmol

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

mariohmol avatar Dec 27 '19 15:12 mariohmol

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

sirbaconjr avatar Dec 28 '19 14:12 sirbaconjr