search

pigor / zaqueu

http://idopter.com/users/paulo-igor/ideas/186-cobrador-online
2 stars 1 forks source link

issue #14, issue #22 #23

Closed GeraldoSequeira closed 10 years ago

GeraldoSequeira commented 10 years ago

issue #14 - Adicionando edição para uma cobrança.

issue #22 - Adicionando mensagens de erros mais descritivas através da internacionalização

Removendo erro ao salvar cobrança sem o campo deadline (undefined method < for nil:NilClass)

Refatoração do form new de collection, adição de mascara para campo deadline.

pigor commented 10 years ago

Geraldo, valeu pelas contribuições eu tomei a liberdade de fazer algumas alterações que eu acredito que deixem o código um pouco melhor. Da uma olhada nas alterações que estão na branch pr/23, principalmente nos commits pigor@0e0dfaa, pigor@5f28cd4 e pigor@6682645 na descrição de cada commit expliquei um pouco do foi feito e por isso fiz os commits separados para cada arquivo...aguardo seus comentários para finalizar o pull request. Mais uma vez obrigado pela contribuição!

GeraldoSequeira commented 10 years ago

Fale Paulo, obrigado pelos comentários e alterações. Prometo não mandar mais código tarde da noite =p.

Sobre essa alteração aqui: pigor@5f28cd4https://github.com/pigor/zaqueu/commit/5f28cd4

Eu tentei fazer com internacionalização mas não funcionou =/. Tentei adicionar linhas de formatação no config/environment.rb e nada!

Vou pesquisar um pouco mais como podemos fazer essa formatação de uma forma mais agradável, sem formatar na mão. Acho que pode ser alguma coisa diferente no rails 4, nos meus aplicativos em produção isso vai de boa:

config/environment.rb

Date::DATE_FORMATS[:default] = '%d/%m/%Y'

Pode finalizar o pull request com suas alterações, e matar o novo branch se for o caso.

Comecei em um pf que vai me consumir o tempo durante a semana. Mas no final de semana vou pegar outras tarefas e mando.

Valeu!

Em 4 de maio de 2014 23:20, Paulo Igor notifications@github.com escreveu:

Geraldo, valeu pelas contribuições eu tomei a liberdade de fazer algumas alterações que eu acredito que deixem o código um pouco melhor. Da uma olhada nas alterações que estão na branch pr/23, principalmente nos commits pigor@0e0dfaa https://github.com/pigor/zaqueu/commit/0e0dfaa, pigor@ 5f28cd4 https://github.com/pigor/zaqueu/commit/5f28cd4 e pigor@6682645https://github.com/pigor/zaqueu/commit/6682645na descrição de cada commit expliquei um pouco do foi feito e por isso fiz os commits separados para cada arquivo...aguardo seus comentários para finalizar o pull request. Mais uma vez obrigado pela contribuição!

— Reply to this email directly or view it on GitHubhttps://github.com/pigor/zaqueu/pull/23#issuecomment-42154841 .

GeraldoSequeira commented 10 years ago

Valeu =)

2014-05-06 20:49 GMT-03:00 Paulo Igor notifications@github.com:

Merged #23 https://github.com/pigor/zaqueu/pull/23.

— Reply to this email directly or view it on GitHubhttps://github.com/pigor/zaqueu/pull/23#event-118484858 .

pigor commented 10 years ago

Da uma olhada no último post no blog da ideia no idopter ;)

On 5/6/14, Geraldo Sequeira notifications@github.com wrote:

Valeu =)

2014-05-06 20:49 GMT-03:00 Paulo Igor notifications@github.com:

Merged #23 https://github.com/pigor/zaqueu/pull/23.

— Reply to this email directly or view it on GitHubhttps://github.com/pigor/zaqueu/pull/23#event-118484858 .

Reply to this email directly or view it on GitHub: https://github.com/pigor/zaqueu/pull/23#issuecomment-42374583

Paulo Igor.

(91) 8114-4916 / (91) 3038-8821 http://about.me/pigodinho

GeraldoSequeira commented 10 years ago

Muito massa, valeu =]

2014-05-06 21:10 GMT-03:00 Paulo Igor notifications@github.com:

Da uma olhada no último post no blog da ideia no idopter ;)

On 5/6/14, Geraldo Sequeira notifications@github.com wrote:

Valeu =)

2014-05-06 20:49 GMT-03:00 Paulo Igor notifications@github.com:

Merged #23 https://github.com/pigor/zaqueu/pull/23.

— Reply to this email directly or view it on GitHubhttps://github.com/pigor/zaqueu/pull/23#event-118484858 .

Reply to this email directly or view it on GitHub: https://github.com/pigor/zaqueu/pull/23#issuecomment-42374583

Paulo Igor.

(91) 8114-4916 / (91) 3038-8821 http://about.me/pigodinho

— Reply to this email directly or view it on GitHubhttps://github.com/pigor/zaqueu/pull/23#issuecomment-42375360 .