Skip to content
This repository was archived by the owner on Apr 3, 2021. It is now read-only.

[WIP] Correção da estrutura atual da aplicação - #23

Draft
acidiney wants to merge 3 commits into
devfrom
feat/better-struture
Draft

[WIP] Correção da estrutura atual da aplicação#23
acidiney wants to merge 3 commits into
devfrom
feat/better-struture

Conversation

@acidiney

Copy link
Copy Markdown
Owner

Motivação

A medida que a aplicação foi crescendo de um simples consultor de nif para um agregador de BI e provincias notei que começou a aparecer tambem as debilidades de não ter uma estrutura definida.

Nessa PR venho com uma proposta de estrutura de forma a melhorar a NON-STRUTURE actual.

O que inclui nessa PR

  • Removi finalmente a rota deprecad nif/:nif
  • Melhorei a estrutura quebrando em pequenos pedaços os código que outra hora estavam ordenados
  • Alterei a rota search/document?type=type&number=number para search/:type/:number [ BREAK CHANGE ]
  • Adicionei slug para as provincias de modo permitir a busca por slug e não ID
  • Alterei a rota provinces/:id para provinces/:slug
  • Adicionei o commit lint e commitzen para padronizar os commits do projecto.
  • Fechei as branchs a única forma de fazer os merge é por PR, de modo a que siga o fluxo do Github flow
  • Removi o redirect para o Github
  • Usar o docs.buscador.ao
  • Adicionar testes aos metodos atuais
  • Adicionar estrutura de github flow com cobertura de testes
  • Adicionar arquivo de contribuidores
  • Criar pacote no NPM para facilitar a integração

@JoseCage @FranciiscoCampos170 @overlineink

Comment thread src/app.controller.ts
res.redirect(303, 'https://github.com/acidiney/buscador-ao')
getHello() {
return {
message: 'Verifique a documentação'

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Acredito que seria melhor manter o redirect por agora até o site para a documentacão ser lancado.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

No caso eu recomendo é reencaminhar para docs enquanto não há alguma página web no domínio principal. Landing page etc.

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

My bad, eu já tenho o sub-dominio preparado.. mas sim, tem que manter o redirect mesmo por agora, vou corrigir isso

@overlineink overlineink Sep 24, 2020

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Já existe alguma preview ou protótipo do site feito?

@acidiney acidiney Sep 25, 2020

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Ainda não, mas deve ser algo simples, usarei algum scaffold

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants