[IMP] point_of_sale: Add QR code payment method for POS#148803
Closed
mhch-odoo wants to merge 7 commits into
Closed
[IMP] point_of_sale: Add QR code payment method for POS#148803mhch-odoo wants to merge 7 commits into
mhch-odoo wants to merge 7 commits into
Conversation
Contributor
|
|
fja-odoo
reviewed
Jan 11, 2024
tong-odoo
requested changes
Jan 11, 2024
mhch-odoo
force-pushed
the
master-point_of_sale-mhch
branch
from
1dea13b to
b5f9be6
Compare
mhch-odoo
force-pushed
the
master-point_of_sale-mhch
branch
3 times, most recently
from
11917a0 to
3f44fa3
Compare
fja-odoo
reviewed
Jan 29, 2024
mhch-odoo
force-pushed
the
master-point_of_sale-mhch
branch
from
3f44fa3 to
df57126
Compare
mhch-odoo
force-pushed
the
master-point_of_sale-mhch
branch
5 times, most recently
from
572b0cb to
64df730
Compare
fja-odoo
force-pushed
the
master-point_of_sale-mhch
branch
from
64df730 to
9e70b68
Compare
C3POdoo
requested review from
a team,
JulienVR and
rhe-odoo
and removed request for
a team
fja-odoo
force-pushed
the
master-point_of_sale-mhch
branch
4 times, most recently
from
b876000 to
0a77009
Compare
Contributor
|
@caburj It seems this PR is breaking L10n tests (cf https://runbot.odoo.com/runbot/build/59658951) |
Contributor
|
@mhch-odoo @caburj staging failed: ci/runbot (view more at https://runbot.odoo.com/runbot/build/59686594) |
fja-odoo
force-pushed
the
master-point_of_sale-mhch
branch
from
ea317ab to
ea3a86d
Compare
As point_of_sale requires a other modules to generate QR Code Payments, this module installs the test dependancies. It runs standard order flow with QR payment methods. 3617200
fja-odoo
force-pushed
the
master-point_of_sale-mhch
branch
from
ea3a86d to
d191ee9
Compare
Contributor
|
It's now passing the l10n test. robodoo rebase-ff r+ |
Contributor
|
Merge method set to rebase and fast-forward. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
8 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
In the accounting module we have the possibility to display a qr code
for payment in an invoice.
These changes aim at enabeling the POS to use that feature to display
payment qr code at checkout.
Currently this doesn't support payment status fetching so the cachier
will have to manually confirm that the payment is received.
We currently suppport all QR from invoicing:
SAPA
SWISS QR
EMV QR
QRIS
task-3617200
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr