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

Corrige interface #196

Merged
merged 4 commits into from
Jan 20, 2021
Merged

Corrige interface #196

merged 4 commits into from
Jan 20, 2021

Conversation

bearkfear
Copy link
Contributor

Uma definição de export só pode ser declarado seguido de um const, function, arrow function entre outros.

Uma definição de export só pode ser declarado seguido de um const, function, arrow function entre outros
As configurações eram validadas, mas no módulo não era especificado que podia fazer uso. Adiciona “interfaces” para configurações com validação das chaves que vão ser inseridas
@bearkfear bearkfear changed the title Definição de export incorreto Corrige interface Nov 19, 2020
@bearkfear
Copy link
Contributor Author

Adicionei as definições mais precisas da “interface” do cep promise de modo que o dev tenha como especificar quais providers quer usar. As configurações eram executadas, mas não especificadas nas “interfaces”. Também adicione validação dos providers que vão ser utilizado, sendo então, um provider declarado não incluído nos AvaliableProviders criando um type error no editor. As definições de providers servem também para auto-complete nos editores com intellisense tais como vscode, intellij...

Comment on lines +14 to +18
type AvaliableProviders =
"brasilapi" |
"correios" |
"viacep" |
"widenet"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Define quais providers podem ser usados

Comment on lines +20 to +24
export interface Configurations {
providers: AvaliableProviders[]
}

export function cep(cep: string | number, configurations: Configurations): Promise<CEP>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Corrige exportação da interface

Copy link
Member

@lucianopf lucianopf left a comment

Choose a reason for hiding this comment

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

❤️ Boooa mano, a gnt adicionou as configs mas não atualizamos a interface 🤦 😂

@lucianopf
Copy link
Member

Mestres, só pra não deixar vcs perdidos abri uma issue com um planinho de ação pra reorganizar o repo dado a migração pra org do BrasilAPI 😬

#197

@lucianopf
Copy link
Member

Muuito obrigado pelo PR e pela paciência @bearkfear!!! 🔝 🚀 ❤️

@lucianopf lucianopf merged commit 5ac4e15 into BrasilAPI:master Jan 20, 2021
@lucianopf
Copy link
Member

Alteração publicada no pacote na versão 4.0.4 ❤️

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