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

Feat: disabled shelter filter #125

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

JuanGustah
Copy link

Descrição:

Essa mudança visa permitir a correção do erro apontado na issue SOS-RS/frontend#178 do front-end. Adicionando um campo enabled na tabela de shelter do banco de dados, e filtrando os abrigos ativos na listagem. Por padrão, abrigos desativados não deverão aparecer, mas caso haja necessidade, pode ser passado a propriedade showDisabled na busca para que eles também sejam retornados.

Arquivos modificados:

prisma/schema.prisma: Adição do campo enabled ao banco de dados.
prisma/migrations/20240516032930_/migration.sql: Migração com o campo novo adicionado ao banco de dados
prisma/dev_dump.sql: Adição do campo enabled ao registro no dump do banco de dados.
src/shelter/shelter.service.ts: Retornar o campo enabled na listagem
src/shelter/ShelterSearch.ts: Filtragem dos abrigos desabilitados, e adição da opção de exibir os desabilitados na listagem
src/shelter/types/types.ts: Tipagem do campo na entidade shelter
src/shelter/types/search.types.ts: Tipagem do campo na busca de abrigos

@JuanGustah
Copy link
Author

JuanGustah commented May 20, 2024

O commit a4be895 inseriu a coluna "actived", no banco de dados , que aparentemente desempenha o mesmo papel da coluna "enabled" desse PR (mesmo que hoje no sistema esse campo não seja utilizada). Por isso, para evitar duplicação de colunas no banco de dados, os trechos de códigos onde referenciava a coluna e os termos "enabled"/"disabled", foram alterados para "actived"/"deactivated".

@rodrigooler
@patrickbathu

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
can-merge enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants