Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: drawing structured addresses #5

Merged
merged 1 commit into from
Sep 11, 2024
Merged

fix: drawing structured addresses #5

merged 1 commit into from
Sep 11, 2024

Conversation

franco
Copy link
Contributor

@franco franco commented Sep 2, 2024

According to the Swiss QR-bill specification, a structured address (type S) has line1 representing the street name and line2 representing the street number. When a structured address is provided, the library currently prints line1 and line2 on two separate lines. However, for type S addresses, line1 and line2 should be printed combined on a single line.

While it is possible to provide the street name and number combined in line1 as a workaround, most e-banking software expects these to be separate. Otherwise, the software may not populate its form correctly, causing unnecessary friction in the payment process.

Screenshot 2024-09-02 at 11 33 30

According to the [Swiss QR-bill specification][1], a structured
address (type S) has `line1` representing the _street name_ and `line2`
representing the _street number_. When a structured address is provided,
the library currently prints `line1` and `line2` on two separate
lines. However, for type S addresses, `line1` and `line2` should
be printed combined on a single line.

While it is possible to provide the _street name_ and _number_ combined
in `line1` as a workaround, most e-banking software expects these to
be separate. Otherwise, the software may not populate its form
correctly, causing unnecessary friction in the payment process.

[1]: https://www.six-group.com/dam/download/banking-services/standardization/qr-bill/ig-qr-bill-v2.2-en.pdf
@mitosch
Copy link
Owner

mitosch commented Sep 11, 2024

Thank you very much for your effort and finding this issue. I will merge it as soon as possible.

BTW: I just saw, starting from Nov 2025 only structured address types are valid. Combined addresses are getting removed:

image

Source: Swiss QR-bill spec, V2.3

@mitosch mitosch merged commit 9d14016 into mitosch:master Sep 11, 2024
5 checks passed
@franco
Copy link
Contributor Author

franco commented Sep 12, 2024

Oh, I wasn't aware that combined addresses are being phased out. Good to know. 🙏🏼

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.

2 participants