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

Listagem de visitantes cadastrados e Filtros para listagens #121

Merged
merged 14 commits into from
Jul 21, 2024

Conversation

thalytalima211
Copy link
Contributor

@thalytalima211 thalytalima211 commented Jul 19, 2024

Introduçao

Administrador agora pode ver a listagem de todos os visitantes cadastrados em um condominio, com filtros para facilitar a pesquisa

Objetivos

  • Listagem com todos os visitantes cadastrados para um condominio a partir da Agenda do dia
  • Autenticacao e autorizacao para a tela
  • Breadcrumbs para as telas criadas
  • Implementacao de filtragem com varios campos para a lista de visitantes de um condo, visitantes de um morador e para registros de entrada

Detalhes

A Agenda agora pode levar para a lista completa de visitantes registrados
Botao na agenda para lista completa

Tela com listagem de todos os registros de visitantes ativos
Listagem completa de visitantes de um condo

A lista de registros agora tem filtros
Lista de Registros de entrada

Lista de visitantes registrados visto pelo morador agora tem filtro
Lista de Meus vistantes registrados

RyanOxon and others added 4 commits July 19, 2024 19:58
…me do visitante, nome do responsavel, e data autorizada para os registros de visitante do morador

Co-authored-by: thalyta lima <[email protected]>
Co-authored-by: Rafael Salgado <[email protected]>
@RyanOxon RyanOxon changed the title Listagem visitantes cadastrados Listagem de visitantes cadastrados e Filtros para listagens Jul 20, 2024
@RyanOxon RyanOxon marked this pull request as ready for review July 20, 2024 20:42
Co-authored-by: thalyta lima <[email protected]>
Copy link
Contributor

@ruliancruz ruliancruz left a comment

Choose a reason for hiding this comment

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

Gostei da implementação da tarefa podendo filtrar por diversos campos ao mesmo tempo. As telas também ficaram bonitas. Com um Vue para buscas dinâmicas ficaria ainda melhor.

Aprovado, mas deixei alguns comentários.

app/controllers/visitor_entries_controller.rb Outdated Show resolved Hide resolved
app/controllers/visitors_controller.rb Show resolved Hide resolved
app/views/visitors/all.html.erb Outdated Show resolved Hide resolved
app/views/visitors/index.html.erb Show resolved Hide resolved
@RyanOxon RyanOxon linked an issue Jul 21, 2024 that may be closed by this pull request
Copy link
Contributor

@Vinigperuzzi Vinigperuzzi left a comment

Choose a reason for hiding this comment

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

Alguns detalhes que como estávamos em call nem cheguei a comentar aqui, falei direto e já foi corrigido.

@RyanOxon RyanOxon merged commit e54d101 into main Jul 21, 2024
2 checks passed
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.

Consultar lista de visitantes e funcionários de moradores
4 participants