diogocezar / pizzaiolo

SlackBoot to report Pull Requests from Github
Apache License 2.0
10 stars 4 forks source link

Criar um sistema de persistência de Dados. #10

Closed diogocezar closed 1 year ago

diogocezar commented 1 year ago

Criar um sistema de persistência de dados utilizando SQLite.

Jott4 commented 1 year ago

Acham que algo assim funciona? @diogocezar @Joao208

image

Jott4 commented 1 year ago

btw o sqlite não suporta enum, então o event_type seria uma string acho

diogocezar commented 1 year ago

Então precisamos mesmo de todos esses relacionamentos?

Pq fico pensando, como vamos cadastrar usuários?

E se nós só armazenássemos os PRs?

diogocezar commented 1 year ago

Se bem que podemos ir adicionando na medida que as PRs chegam né! É acho que faz sentido sim man!

Jott4 commented 1 year ago

Isso, acho legal a table de pull Request ter também um status, pra ficar mais fácil na consulta

diogocezar commented 1 year ago

Boa! Acho que pode mandar bala lah no PR ;) pq o próximo passo é tirar o in_memory e substituir por esse esquema ai...

Joao208 commented 1 year ago

Onde vamos salvar a qual mensagem o PR pertence para fazermos atualizações? @Jott4

Jott4 commented 1 year ago

Bom ponto @Joao208 , talvez adicionamos a table Messages que tem relação 1-1 com Pull Request?

diogocezar commented 1 year ago

Pelo que entendi ele usa o timestamp né?

Joao208 commented 1 year ago

Isso @diogocezar, ts que vem no retorno da mensagem, podemos adicionar sim @Jott4. Não entendi então o que seria a tabela Event, mas com Menssage funciona d mais.

Joao208 commented 1 year ago

Pois é praticamente isso que a in memory db está fazendo hoje

Jott4 commented 1 year ago

A Event seria como uma tabela log saca?

Toda vez que o usuário criar, aprovar, recusar uma PR, vai logar na tabela o user_id, a pull_request_id e a ação que o usuário fez

Joao208 commented 1 year ago

Disponibilizar uma documentação do banco de dados, nomes e o que é cada tabela, o que acha @Jott4?

Jott4 commented 1 year ago

É uma boa

diogocezar commented 1 year ago

Finalizada com a implementação #27