claudep / swiss-qr-bill

A Python library to generate Swiss QR-bill payment slips

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

amount, pcode, ref_number, extra_infos

noem6942 opened this issue · comments

Proposals:

  • type(amount) should be float not str
  • rename pcode to postal_code as used in common
  • rename ref_number to reference_number (as in SIX documentation)
  • rename extra_infos to additional_information (as in SIX documentation)
  • implement Billing Information (to my understanding it's approved by 1.1.2021)

Would you like to submit patches? In that case, please one change per commit (could be one or several pull requests). The last point probably deserves a separate issue.

Point 1 to 4 is trivial to implement without any backwards compatibility - but do you really want this?

Sure, it looks like proper cleanup. It might be a bit less trivial if we want to keep compatibility with old names at entry points (script parameters and QRBill.__init__).

type(amount) should be float not str

this is a bad idea.