Retirar texto hardcoded

garoa / telemob

Telemob: para ajudar a sociedade civil a pressionar seus deputados e senadores no Congresso Nacional

https://garoa.net.br/wiki/Telemob

GNU General Public License v3.0

44 stars 10 forks source link

Open deniscostadsc opened 8 years ago

deniscostadsc commented 8 years ago

Todos os textos devem ser salvos no banco. Isso permite atualizar os textos sem precisar mudar o código e fazer um novo deploy.

[x] Texto do template de telegrama/email
[x] Texto dos argumentos
[ ] Os termos específicos para deputados (O telemob deve ser genéricos para qualquer esfera política)
[ ] O endereço deve ser genéricos para qualquer lugar (Hoje é sempre de Brasília)

deniscostadsc commented 8 years ago

Comecei a fazer essa issue. Já retirei o texo do template que é enviado por email. Isso está sendo feito nessa branch https://github.com/garoa/telemob/tree/textos-hardcoded

deniscostadsc commented 8 years ago

Agora estou pensando em como refatorar a parte de coisas a dizer. Acho que existem dois possíveis formas que exigem pouco esforço:

Uma é criar um modelo para o argumento com uma chave estrangeira para a campanha. Aí vc lista todos os argumentos que apontam para a campanha.
O outro é salvar todo o texto na própria campanha. Aí, talvez precisaremos salvar o texto com algum tipo de formatação, para fazer os bullets para cada argumento.

Eu tenho uma tendência a escolher a primeira opção.

O que acha @vtrbtf e @ramalho? Vocês vêm outras possibilidades mais simples de fazer isso?

vtrbtf commented 8 years ago

+1 para criar o modelo separado :)

vtrbtf commented 8 years ago

Duvida:

Hoje o endereço de brasilia é hardcoded, mas acho que é nem seria necessario, pq no dump do xls o endereço ja tem as informações de Brasilia.

Confere, @ramalho ?

deniscostadsc commented 8 years ago

Hoje o endereço de brasilia é hardcoded, mas acho que é nem seria necessario, pq no dump do xls o endereço ja tem as informações de Brasilia.

Precisamos pensar num jeito de formatar o endereço e que seja genérico para qualquer lugar.

deniscostadsc commented 8 years ago

Vou fazer o pull request apenas com dois dos 4 passos. Acho que avançar mais que isso sem teste vai ser difícil.