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

feat: implementando testes unitarios e mascara cnpj #17

Merged
merged 1 commit into from
Feb 1, 2019

Conversation

FabaoAnalista
Copy link
Member

Inicio implementação tarefa.

@FabaoAnalista FabaoAnalista changed the base branch from master to develop January 31, 2019 23:50
@edoura edoura merged commit eb77ad8 into develop Feb 1, 2019
@edoura edoura deleted the feature/validarCnpj branch February 1, 2019 16:50
@edoura
Copy link
Member

edoura commented Feb 1, 2019

Boa @FabaoUfop, aprovado o PR, entretanto os próximos por favor se atente a manter o padrão na descrição do commit e referenciar a issue a qual está trabalhando em questão, ok? No caso desta é a issue #14, certo?
Dessa maneira acompanhamos naturalmente as atividades que vieram acontecendo la no corpo e comentários na issue.

Para mantermos uma organização, vamos cancelar os PRs caso não venham dessa maneira 👍
Fiquem cientes também manos @JordanPena, @verneck.

@edoura edoura added this to the Sprint 1 milestone Feb 6, 2019
@edoura edoura changed the title implementando testes unitarios e mascara cnpj feat: implementando testes unitarios e mascara cnpj Feb 13, 2019
Copy link
Member

@edoura edoura left a comment

Choose a reason for hiding this comment

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

Ok @FabaoUfop, fiquemos atento, como já conversamos, nos padrões da criação das branchs para executar a tarefa e nas descrições dos commits e pull requests.
Outro quesito, sugerido ali embaixo no arquivo src/app/home/home.component.html, é alterar o placeholder. ok? 👍

@JordanPena
Copy link
Member

@FabaoUfop , favor verificar as alterações q não foram submetidas.. faltou a alteração no package.json contendo a dependencia que esta sendo importada e os arquivos referentes ao ComponentCnpj, que tambem está sendo importado (e não são encontrados) no app.module.

Copy link
Member

@edoura edoura left a comment

Choose a reason for hiding this comment

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

Com inconsistências, porem já acertados em commits posteriores

@FabaoAnalista
Copy link
Member Author

Incosistências removidas , continuando atividade de validação cnpj (mascara). @edoura favor dividir tarefa em validação CNPJ (mascara) e validação da natureza juridica .

@edoura
Copy link
Member

edoura commented Feb 22, 2019

Incosistências removidas , continuando atividade de validação cnpj (mascara). @edoura favor dividir tarefa em validação CNPJ (mascara) e validação da natureza juridica .

@FabaoUfop já está assim 👍
Como pode acompanhar lá na issue #14, onde até vi um tick em uma das tasks 😉
Obrigado pela participação e colaboração.

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.

4 participants