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

Feature/info #4

Merged
merged 3 commits into from
Jun 26, 2023
Merged

Feature/info #4

merged 3 commits into from
Jun 26, 2023

Conversation

yxuo
Copy link
Contributor

@yxuo yxuo commented Jun 23, 2023

Mudanças

  • 🎁 Nova rota de API /api/info

    Contém configurações e flags para testar correções e melhorias entre os usuários.
    O frontend fará uso dessas informações para implementação.

Campos

  • id
  • name
  • value
  • version

    O motivo é separar por versão caso necessário.

Por agora entende-se que não há necessidade de adicionar complexidade como data de criação, edição, ou histórico de edição.

Lógica usada

  • A partir de um objeto com dados, gera o seed no banco de forma automática.

    O boilerplate usa vários comandos para gerar cada ocorrência (exemplo: role-seed.service.ts:15-28).
    Contudo, para o Info isto foi feito de forma automatizada no seed service (info-seed.service.ts).

    A vantagem de gerar o seed no banco é permitir que se altere os dados sem precisar subir uma nova release da API.
    Ainda sim, qualquer alteração pode ser feita no arquivo de seed sem problemas.

  • Ao entrar em /api/info a API mostra todas as configurações e flags.

  • Para ver flags relacionadas à versão atual da API basta entrar em, /api/v1, por exemplo.

    Isto mostra flags e configurações para v1 e também aqueles que não possuem versão definida, pois são considerados globais, valem para qualquer versão.

@yxuo yxuo requested a review from rivailruiz June 23, 2023 18:03
@yxuo yxuo linked an issue Jun 23, 2023 that may be closed by this pull request
8 tasks
@rivailruiz rivailruiz merged commit 2741f43 into main Jun 26, 2023
2 checks passed
@rivailruiz rivailruiz mentioned this pull request Jun 26, 2023
@yxuo yxuo deleted the feature/info branch June 26, 2023 12:20
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.

Criar rota de appInfo para receber as configurações do back
2 participants