search

DadosAbertosDeFeira / maria-quiteria

Backend para coleta e disponibilização dos dados 📜
https://mq.dadosabertosdefeira.com.br/painel
MIT License
166 stars 69 forks source link

Adicionar __str__ para agenda e lista de presença dos vereadores #177

Closed anapaulagomes closed 3 years ago

anapaulagomes commented 4 years ago

Dessa forma, podemos ter um título melhor ao clicar na página de registros desses dois modelos (CityCouncilAgenda e CityCouncilAttendanceList).

Exemplo ao clicar na presença de um vereador:

presenca

Para reproduzir esse bug basta ter registros nos modelos citados e clicar em um deles. A barra deve mostrar informações amigáveis (geralmente já disponível no __repr__).

KauanAmarante commented 4 years ago

Oi, tudo bem? Posso puxar essa issue ?

anapaulagomes commented 4 years ago

Manda ver, @KauanAmarante! 🚀

elicaciocf commented 3 years ago

Kauan, boa noite!

Ainda está trabalhando nessa issue, posso puxar?

elicaciocdefarias commented 3 years ago

@anapaulagomes,

Olha como o breadcrumbs do admin aparece

issue_177_1

issue_177_2

Não consigo acessar os models, tu pode me ajudar?

anapaulagomes commented 3 years ago

Claro! Dá uma olhada na lista de presença no /painel (você deve estar no /admin, certo?). @elicaciocdefarias

elicaciocdefarias commented 3 years ago

@anapaulagomes,

Deu certo!!! Valeu

elicaciocdefarias commented 3 years ago

@anapaulagomes,

O que você acha?

issue_177_3

issue_177_4

anapaulagomes commented 3 years ago

Está ficando ótimo, @elicaciocdefarias! O que acha de ao invés de ter a PK ter a data? Além disso, a Lista de Presença é uma para cada vereador. Talvez seja melhor ter para ela a data e o nome do vereador também. Tipo "Agenda para 01/10/2020" e "Presença em 01/10/2020 - Fulano de Tal". Acho que assim fica mais humano. Faz sentido?

elicaciocdefarias commented 3 years ago

Claro! Faz todo sentido.

elicaciocdefarias commented 3 years ago

@anapaulagomes ,

Para o model CityCouncilAgenda, acredito que podemos usar o título, já que ele traz a data e o tipo do evento. ficaria assim: "Agenda para ORDEM DO DIA - 09 DE DEZEMBRO DE 2020", que acha?

anapaulagomes commented 3 years ago

Ótimo! Mas confere se todos trazem a data no título. Se sim, manda ver. 👍🏽 @elicaciocdefarias