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

Padronizar o idioma do código #3

Open
juaoantonio opened this issue Sep 11, 2023 · 4 comments
Open

Padronizar o idioma do código #3

juaoantonio opened this issue Sep 11, 2023 · 4 comments
Labels
good first issue Good for newcomers

Comments

@juaoantonio
Copy link

Muitas das vezes, há uma mistura entre inglês e português, talvez se proveitoso manter tudo em inglês, facilitando a entrada de novos colaboradores.

@Victoreisdavid
Copy link
Owner

Boa noite!

Eu queria manter tudo em inglês, mas a Api dos dados abertos retorna tudo em português, isso me deixou bastante confuso durante o decorrer do projeto 😅.

Padronizar o idioma é uma das coisas que quero fazer, inclusive, se quiser ajudar, será bem vindo! ❤

@wellwelwel
Copy link

wellwelwel commented Oct 4, 2023

Olha eu aqui outra vez 😂

Acredito que uma forma super prática, seja criar uma abstração de API.
Dessa forma, independente do que a API retorne, a abstração deixa tudo no seu padrão (em inglês, por exemplo) e o projeto passa a chamar os resultados da API a partir dessa abstração ao invés da API de fato.

@Victoreisdavid, você poderia adicionar a label good first contribution/issue e fixar esse Issue no projeto. Assim, todo mundo que entrar na aba Issues vai ver, aumentando a change de uma contribuição 🚀


Só uma observação, tudo que passei aqui são só sugestões 🩵

@wellwelwel
Copy link

wellwelwel commented Oct 4, 2023

Um exemplo bem simples de como isso seria na prática, onde supostamente uma API de CEP retorne as propriedades em português:

export const API = {
  street: ORIGINAL_API.rua,
  neighborhood: ORIGINAL_API.bairro,
  // ...
}

Dessa forma, quando você for utilizar os dados, basta chamar pela sua API:

import { API } from './api.js';

console.log(
  API.street,
  API.neighborhood,
);

@Victoreisdavid Victoreisdavid added the good first issue Good for newcomers label Oct 4, 2023
@Victoreisdavid
Copy link
Owner

Olha eu aqui outra vez 😂

Acredito que uma forma super prática, seja criar uma abstração de API. Dessa forma, independente do que a API retorne, a abstração deixa tudo no seu padrão (em inglês, por exemplo) e o projeto passa a chamar os resultados da API a partir dessa abstração ao invés da API de fato.

@Victoreisdavid, você poderia adicionar a label good first contribution e fixar esse Issue no projeto. Assim, todo mundo que entrar na aba Issues vai ver, aumentando a change de uma contribuição 🚀

Só uma observação, tudo que passei aqui são só sugestões 🩵

Olá Well! Seja bem vindo!

Abstração de API parece ser uma boa ideia, tenho em mente algumas mudanças para fazer no projeto, uma delas inclui manter meu próprio banco de dados atualizado periodicamente, porque o Dados Abertos sempre tá com problemas de estabilidade. Dessa forma, o funcionamento do FiscalizaJa independe da disponibilidade do Dados Abertos.

Farei com certeza algo assim, se não vai dificultar a manutenção do código no futuro.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

3 participants