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

[13.0] [ADD] l10n_ch_qr_bill_scan #553

Open
wants to merge 9 commits into
base: 13.0
Choose a base branch
from

Conversation

yvaucher
Copy link
Member

@yvaucher yvaucher commented Jun 23, 2020

A new module to scan QR-bill or to import PDF files

This module is based on account_invoice_import available here:
OCA/edi#213

2020-06-23-182753_1013x572_scrot

2020-06-23-182820_1030x901_scrot

TODO:

  • Parse Swico standard
  • read QR from PNG canceled, due to the swiss cross it is complicated to read any size QR-Code
  • read QR from PDF

⚠️ due to the v13 supplier_rank field, it can only matches a supplier if this partner had already one Vendor bill posted.
I extended the wizard to be able to set the partner to solve this issue.

@yvaucher yvaucher force-pushed the 13.0-add-l10n_ch_qr_bill_scan branch 2 times, most recently from 3fad455 to c0c9f7b Compare June 24, 2020 08:14
@yvaucher yvaucher force-pushed the 13.0-add-l10n_ch_qr_bill_scan branch from 67400a1 to c255782 Compare June 25, 2020 14:33
@yvaucher yvaucher marked this pull request as ready for review June 25, 2020 17:46
@yvaucher yvaucher force-pushed the 13.0-add-l10n_ch_qr_bill_scan branch from 6b56531 to 6d55a1e Compare June 26, 2020 08:38
@i-vyshnevska i-vyshnevska force-pushed the 13.0-add-l10n_ch_qr_bill_scan branch 2 times, most recently from 6be66c3 to d926f16 Compare February 17, 2021 15:34
@i-vyshnevska i-vyshnevska force-pushed the 13.0-add-l10n_ch_qr_bill_scan branch from d926f16 to 9806df5 Compare February 17, 2021 15:37
Comment on lines 184 to 188
def test_scan_QR_swico(self):
self.assertTrue(False)

def test_scan_QR_wrong_swico(self):
self.assertTrue(False)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤨 ❓

Copy link
Contributor

@Martronic-SA Martronic-SA left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested and working as expected

@github-actions
Copy link

github-actions bot commented Mar 6, 2022

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Mar 6, 2022
@github-actions github-actions bot closed this Apr 10, 2022
@yvaucher yvaucher removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Jun 15, 2022
@yvaucher yvaucher reopened this Jun 15, 2022
@yvaucher yvaucher added the no stale Use this label to prevent the automated stale action from closing this PR/Issue. label Jun 15, 2022
@bosd
Copy link

bosd commented Feb 20, 2023

@yvaucher Can you force push/rebase to retrigger runbot creation?

@ecino
Copy link

ecino commented Nov 30, 2023

What's the status of this project? I think we should consider either make the last steps for merging or close this pull request. Thanks.

@ecino
Copy link

ecino commented Dec 18, 2023

@yvaucher Can you check my last comment? I think there are also other PR for migration, it would be good if this module exists officialy in the branches.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no stale Use this label to prevent the automated stale action from closing this PR/Issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants