Remover django_municipios

[filipecifali]

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]

Vou trabalhar nesta issue

[lskbr]

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]

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

[lskbr]

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 < [email protected]> 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 <#192 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAEXJW4GKR3XHU7KIFKLWX3SOJZG7ANCNFSM4JF5APKA> .

[filipecifali]

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

[lskbr]

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 < [email protected]> 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 <#192 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAEXJW6S5U6GNKYIGHJZRGLSOJ4KJANCNFSM4JF5APKA> .

[cuducos]

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]

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

[mazulo]

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]

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:

• ter Simplifica dependências e instação #226 e Atualiza para Django 2.2 #228 mergeados
• criar PR para Juntar settings.py #227
• aí sim chega essa issue
• e, depois, migrar para Django 3

Faz sentido?

[rougeth]

Faz demais @cuducos :)