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

Add a new menu for the admin in order to check the quality of the import #156

Merged
merged 1 commit into from
Dec 15, 2015
Merged

Conversation

loikki
Copy link
Contributor

@loikki loikki commented Dec 11, 2015

In Sponsorship->Configuration->Test Imports admin can do an import
(with an additionnal csv) that will compare the values obtained and
the value in the csv.

@loikki
Copy link
Contributor Author

loikki commented Dec 11, 2015

No need to review yet, I still want to add something to the preview

class TestImportLetters(models.TransientModel):
""" Class testing the letters import by highlighting the keypoints of
the import (image used, keypoints found for the templates, showing only
the line not well detected, ...)
Copy link
Member

Choose a reason for hiding this comment

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

Avoid ... in comments

error = error + 'template,\n'

# second: qr codexb
partner_id = env['res.partner'].search([
Copy link
Member

Choose a reason for hiding this comment

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

rename to partner_detected

from glob import glob
from wand.image import Image
from collections import namedtuple

from openerp import _
Copy link
Member

Choose a reason for hiding this comment

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

Add import exceptions.Warning

@ecino
Copy link
Member

ecino commented Dec 15, 2015

Languages cut result should be shown even if no language is detected (if template is detected)

Fix: #150
In Sponsorship->Configuration->Test Imports the admin can do an import
(with an additionnal csv) that will compare the values obtained and
the value in the csv.
@ecino
Copy link
Member

ecino commented Dec 15, 2015

👍

ecino added a commit that referenced this pull request Dec 15, 2015
Add a new menu for the admin in order to check the quality of the import
@ecino ecino merged commit 07a2539 into CompassionCH:sbc-devel Dec 15, 2015
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