search

apyb / associados

Controle de associados a Associação PythonBrasil
https://associacao.python.org.br/
105 stars 50 forks source link

Remover django_municipios #192

Open filipecifali opened 5 years ago

filipecifali commented 5 years ago

A informação não é usada para nada e apenas aumentou o tamanho da base de dados.

Podemos adicionar no futuro CASO seja necessário e com um propósito claro.

O custo de ter adicionado isso, desde sua adição, o Tier de banco de dados da Heroku deixou de ser gratuito, gerando cobrança adicional de 7 dólares ao mês.

pythrick commented 5 years ago

Vou trabalhar nesta issue

lskbr commented 4 years ago

A ideia de ter os nomes padronizados é que não era possível gerar estatísticas por cidade/estado. Cada pessoa escrevia de um jeito. Se não interessa em gerar estatística alguma, ai sim, não tem importância.

filipecifali commented 4 years ago

Definitivamente não interessa (porque nunca foi usada desde sua concepção) e só gerou dependências.

lskbr commented 4 years ago

Talvez o caminho seja sair do Heroku, $7 por mes so pro banco de dados nao faz sentido.

Le jeu. 5 nov. 2020 à 10:57, Filipe Cifali Stangler < notifications@github.com> a écrit :

Definitivamente não interessa (porque nunca foi usada desde sua concepção) e só gerou dependências.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/pythonbrasil/associados/issues/192#issuecomment-722271299, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAEXJW4GKR3XHU7KIFKLWX3SOJZG7ANCNFSM4JF5APKA .

filipecifali commented 4 years ago

Anteriormente a esta adição também não precisavamos pagar, então qualquer um dos casos resolve a issue.

lskbr commented 4 years ago

Depende do que vamos querer fazer com o software. Se não serve pra nada, só para coletar os pagamentos de vez em quando, ok. Mas não podemos ficar com uma limitação no banco de dados. Seria legar ter um plano para essa aplicação e discutir antes de decidir as coisas. Eu enviei um email com mais detalhes para lista de associados. Infelizmente não anotamos em lugar nenhum o esquema da estatística e o aplicativo está bem velhinho :-D Um redesign pode ser necessário, mas precisamos saber o queremos fazer e onde queremos chegar. Se o atual já está bom, podemos focar só em atualizar o Django e corrigir bugs, por exemplo. Quanto ao Django Municipios, nada contra. Talvez tenha outra solução que permita atingir o mesmo objetivo. Nem sei se a base está atualizada, acho que foi o Cadu que criou o Django Municipios.

Le jeu. 5 nov. 2020 à 11:23, Filipe Cifali Stangler < notifications@github.com> a écrit :

Anteriormente a esta adição também não precisavamos pagar, então qualquer um dos casos resolve a issue.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/pythonbrasil/associados/issues/192#issuecomment-722285644, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAEXJW6S5U6GNKYIGHJZRGLSOJ4KJANCNFSM4JF5APKA .

cuducos commented 2 years ago

Alguém pode atualizar o estado dessa issue?

Aparentemenye #199 seria uma solução, o code review pareceu positivo, mas o PR foi fechado ao invés de mergeado — aí me perdi.

filipecifali commented 2 years ago

@pythrick @mazulo vocês sabem informar o status?

mazulo commented 2 years ago

O status foi de que a remoção (pelo menos a proposta na época) seria complicada demais. tanto é que até criamos uma versão staging da app pra testar, e envolvia muita coisa (como até entrar no dbshell e remover algumas tabelas na mão). Acredito que agora com novos conhecimentos possamos rever como fazer isso.

TL;DR: não foi resolvido ainda

cuducos commented 2 years ago

Perfeito. Me empolgo demais em remover isso : )

Não estou dizendo que estou fazendo — acho que qualquer pessoa interessada pode chegar e fazer. Mas, gerenciando meu tempo de colaboração, estou colocando em uma lista de coisas que quero fazer:

Faz sentido?

rougeth commented 2 years ago

Faz demais @cuducos :)