wcustodio / pagseguro-node

Integração PagSeguro UOL com checkout transparente para Node.js
48 stars 21 forks source link

Rename `qtde` to `quantity` ? #13

Open hems opened 1 year ago

hems commented 1 year ago

In order to be consistent with the rest of the code ( which is written in english ) the parameter should be renamed from 'qtde' to 'quantity'.

Also in general it's not really a good idea to abbreviate variable names ( :

https://github.com/wcustodio/pagseguro-node/blob/64c87a98fdba460b3b14451256aa01abcbef8880/src/pagseguro.js#L76

thanks a lot for the work put in thiis library, this is very useful!