Skip to content

docs: pass payment amount by keyword - #24

Open
awais-ur-rehman wants to merge 1 commit into
ViktorStiskala:mainfrom
awais-ur-rehman:feat/fix-example-payment-amount
Open

awais-ur-rehman wants to merge 1 commit into
ViktorStiskala:mainfrom
awais-ur-rehman:feat/fix-example-payment-amount

Conversation

@awais-ur-rehman

Copy link
Copy Markdown

Summary

The README passes 400.56 as the second positional argument, which is now bic. Both examples therefore put the amount into the account's BIC suffix and omit the payment amount. Pass amount=400.56 in the QR and standalone SPAYD examples.

Fixes #23.

Breaking Changes

None.

Testing

  • Executed both README Python blocks in a temporary directory with PNG support installed. Before the change, both payloads contained ACC:...+400.56 and no AM field; afterward both contain AM:400.56 and leave bic unset.
  • The full image example generated SVG and PNG files successfully.
  • pytest -q: 71 passed.
  • ruff check . and ruff format --check qrplatba tests passed.
  • Whole-tree ruff format --check . reports existing README string-quote formatting differences, also reproduced on the unmodified README.
  • git diff --check passed.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Example code is broken

1 participant