Skip to content

Criação do endpoint para busca de cidades dos abrigos - #82

Merged
fagundesjg merged 5 commits into
SOS-RS:developfrom
AndersonCRocha:feat/shelter-cities-endpoint-and-searching
May 13, 2024
Merged

Criação do endpoint para busca de cidades dos abrigos#82
fagundesjg merged 5 commits into
SOS-RS:developfrom
AndersonCRocha:feat/shelter-cities-endpoint-and-searching

Conversation

@AndersonCRocha

@AndersonCRocha AndersonCRocha commented May 12, 2024

Copy link
Copy Markdown
Contributor

Depende de #80

Este PR contém:

  • Remoção de arquivos vazios não utilizados
  • Criação do endpoint GET /shelters/cities que retorna todas as cidades cadastradas com a quantidade de abrigos correspondentes
  • Inclusão do filtro por cidades no endpoint GET /shelters
  • Remoção de cast duvidoso no query param parseado com a lib qs (as unknown as IFilterFormProps) para validação e conversão do tipo via Zod
  • Melhorias de legibilidade em algumas lógicas de aplicação dos filtros da classe ShelterSearch
@antoniolucasnobar

Copy link
Copy Markdown

Aparentemente, esse PR conflita parcialmente com o #57 Seria interessante verificar qual vai prosseguir ou como conciliar os 2, já que aquele traz uma estratégia para o preenchimento dos dados dos abrigos a partir de uma API, o que eu não vi nesse PR.

@fagundesjg
fagundesjg merged commit 9362f4c into SOS-RS:develop May 13, 2024
filipepacheco pushed a commit to filipepacheco/sos-rs-backend that referenced this pull request May 15, 2024
Co-authored-by: Pedro Perrone <psperrone@hotmail.com>
filipepacheco pushed a commit to filipepacheco/sos-rs-backend that referenced this pull request May 15, 2024
Co-authored-by: Pedro Perrone <psperrone@hotmail.com>
HelioDantas pushed a commit to HelioDantas/backend that referenced this pull request May 16, 2024
Co-authored-by: Pedro Perrone <psperrone@hotmail.com>
@AndersonCRocha
AndersonCRocha deleted the feat/shelter-cities-endpoint-and-searching branch May 16, 2024 03:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

5 participants