search

eventick / pagseguro-transparente

Gem para a utilização do novo checkout transparente do Pagseguro
18 stars 12 forks source link

Configuração via YAML #16

Open danieldocki opened 9 years ago

danieldocki commented 9 years ago

@cirdes

Adicionei generators para o Rails Initializer e YAML. Seria isso? ou pode melhorar?

cirdes commented 9 years ago

@danieldocki, muito bom o initializer.

Você poderia fazer o Squashing dos commits: http://guides.rubyonrails.org/contributing_to_ruby_on_rails.html#iterate-as-necessary

Além disso adiciona uma linha em branco no final dos arquivos como o Git pede.

danieldocki commented 9 years ago

@cirdes @brunoluigi Obrigado! Farei as mudanças :)

danieldocki commented 9 years ago

@cirdes @brunoluigi Seria isso? Apanhei um pouquinho para fazer Squashing kkk

cirdes commented 9 years ago

@danieldocki, o squashing é isso mesmo. Você poderia fazer um rebase com o master? Adicionei bastante coisa no README.

Acho que a gente deveria remover o "adapter_javascript_url" do initializer, é melhor incentivar que as pessoas baixam e coloquem na pasta de vendor. Mas acho importante deixar claro que é preciso incluir a lib.

Você pode fazer essas alterações para gente mergiar?

danieldocki commented 9 years ago

@cirdes, e se o pagseguro mudar algo no javascript ele ficará com uma versão desatualizada, mas ela realmente não faz parte da api transparente mas é necessário, acho que uma menção no README já basta.

cirdes commented 9 years ago

@danieldocki, é verdade.