search

pagarme / node-boleto

Boleto generator in Node.js
MIT License
3 stars 2 forks source link

Allow custom assets to images #20

Closed ortense closed 4 years ago

ortense commented 7 years ago

Custom assets are useful for avoiding additional requests to pagar.me host.

theiostream commented 7 years ago

Obrigado pelo PR. Para que possamos aceitar a change, você precisa seguir as seguintes orientações:

  1. Por favor squasheie os commits ef349a3 e 81bf30a
  2. Por favor inclua na mensagem de commit exatamente o porquê da feature e detalhe melhor a nova interface do que ele faz.
  3. Fora o que tem no ponto 2, melhore o título dos commits. Sugiro, respectivamente, "Allow custom assets for rendering" e "Add E2E tests for custom assets".
  4. Há algumas irregularidades na indentação do arquivo .ejs. É uma falha do nosso linter não apontar esse tipo de coisa, mas tente manter o padrão de espaços que ele já tinha (ou, se você quiser nos ajudar, configurar alguma coisa de fazer problemas de intentação falharem nos testes).
  5. Documente no README as suas mudanças.
lucianopf commented 4 years ago

Devido a inatividade fecharei o PR.