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

hotfix/validate-proviers #198

Merged
merged 1 commit into from
Jun 20, 2021
Merged

hotfix/validate-proviers #198

merged 1 commit into from
Jun 20, 2021

Conversation

bearkfear
Copy link
Contributor

A atual implementação está validando apenas o primeiro provider do array de providers passado por parâmetro. Removendo o return, todos os providers serão validados.

@bearkfear
Copy link
Contributor Author

for (const provider of providers) {
if (!availableProviders.includes(provider)) {
throw new CepPromiseError({
message: 'Erro ao inicializar a instância do CepPromise.',
type: 'validation_error',
errors: [
{
message:
`O provider "${provider}" é inválido. Os providers disponíveis são: ["${availableProviders.join('", "')}"].`,
service: 'providers_validation'
}
]
})
}
}
}

@lucianopf
Copy link
Member

Você tem toda razão @bearkfear !! 😱

Só pra vc tomar ciência, eu to preparando o terreno dado a migração de orgs e ainda tem algumas pendências pra deixar tudo redondinho 😢

Perdão pela demora pra andar com esse PR mas prometo que vou dar a atenção devida!!! ^^

Um pouquinho mais de contexto pra caso queira acompanhar o progresso ^^

#197

@bearkfear
Copy link
Contributor Author

Tranquilo, eu só tenho umas horinhas na noite para isso tmb! abraço!

A atual impe mentação está validando apenas o primeiro provider do array de providers passado por parâmetro. Removendo o return, todos os providers serão validados.
@lucianopf
Copy link
Member

@bearkfear mil perdões pela demora desse PR mestre! 😢
Esse PR é super importante e simples que não tem desculpas pra gente ter deixado tanto tempo parado =/

Muuuito obrigado pelo fix e pela paciencia manão!! ❤️ 🚀 😍

@lucianopf lucianopf merged commit f0a2f47 into BrasilAPI:master Jun 20, 2021
@bearkfear
Copy link
Contributor Author

Tranquilo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants