Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Retirar texto hardcoded #38

Open
2 of 4 tasks
deniscostadsc opened this issue Jan 30, 2016 · 6 comments
Open
2 of 4 tasks

Retirar texto hardcoded #38

deniscostadsc opened this issue Jan 30, 2016 · 6 comments

Comments

@deniscostadsc
Copy link
Member

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

  • Texto do template de telegrama/email
  • 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
Copy link
Member Author

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
Copy link
Member Author

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
Copy link
Contributor

vtrbtf commented Feb 3, 2016

+1 para criar o modelo separado :)

@vtrbtf
Copy link
Contributor

vtrbtf commented Feb 3, 2016

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
Copy link
Member Author

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
Copy link
Member Author

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants