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

Função codPostalVal() implementada #16

Merged
merged 1 commit into from
Jul 14, 2021

Conversation

pypas
Copy link
Contributor

@pypas pypas commented Jul 14, 2021

Referente à issue: #13

O que foi implementado? funçao codPostalVal() para validação de códigos postais, seguindo o padrão de TDD.

@pypas
Copy link
Contributor Author

pypas commented Jul 14, 2021

OBS: é minha primeira tentativa de contribuição para um projeto open source, peço desculpas de antemão se tiver esquecido de algo e fico à disposição para qualquer correção que seja necessária 😅

@EmanuelGF
Copy link
Owner

OBS: é minha primeira tentativa de contribuição para um projeto open source, peço desculpas de antemão se tiver esquecido de algo e fico à disposição para qualquer correção que seja necessária 😅

Oi @pypas !
Sem qualquer problema, amanhã vou ver o teu PR e dou-te feedback.
Muito obrigado pelo teu interesse no projeto e contribuição 😉

Copy link
Owner

@EmanuelGF EmanuelGF left a comment

Choose a reason for hiding this comment

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

Bom dia @pypas !
Para começar está perfeito. 🎉

@EmanuelGF EmanuelGF merged commit 4cbf1fc into EmanuelGF:main Jul 14, 2021
@EmanuelGF
Copy link
Owner

Os códigos postais tem na realidade maior complexidade. Vou ter de encontrar uma lista com os códigos postais usados em Portugal para adicionar essa validação. Mas por agora está ótimo.
Podes ir dando uma olhadela ao Issue #13 para depois se quiseres adicionar essa validação.
Parabéns pela tua primeira contribuição!! 🥇🎉

Se fores do Brasil e quiseres adicionar as validações do Brasil podes criar um novo issue com a validação e testes a adicionar e trabalhar à vontade. 😉

@pypas mais uma vez agradeço a ajuda e estarei disponível caso tiveres duvidas.

@pypas pypas deleted the feat/cod-postal-val branch July 14, 2021 12:38
@pypas
Copy link
Contributor Author

pypas commented Jul 14, 2021

Muito obrigada @EmanuelGF!

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.

2 participants