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

7.0 fattura pa backport #659

Closed
wants to merge 181 commits into from
Closed

Conversation

OmniaGit
Copy link

The in and out of the electronic invoice is now working and ready to be merged

Copy link
Contributor

@sergiocorato sergiocorato left a comment

Choose a reason for hiding this comment

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

@dsmerghetto Puoi correggere il .travis.yml con pyxb 1.2.5?

from openerp.osv import orm
from openerp.addons.l10n_it_fatturapa.bindings.fatturapa_v_1_1 import (
from openerp.osv import fields
from openerp.addons.l10n_it_fatturapa.bindings.fatturapa_v_1_2 import (
FatturaElettronica,
FatturaElettronicaHeaderType,
DatiTrasmissioneType,
Copy link
Contributor

Choose a reason for hiding this comment

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

per curiosità, perchè sono stati tolti alcuni import?

Copy link
Author

Choose a reason for hiding this comment

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

abbiamo voluto allinearlo a quello di 10.. non penso ci siano controindicazioni ..

Copy link
Contributor

Choose a reason for hiding this comment

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

a dir la verità nella 10 ce ne sono di più, vedi

Copy link
Author

Choose a reason for hiding this comment

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

nei prossimi giorni mergiamo gli ultimi commit di 10 cosi allineiamo il tutto definitivamente

@primes2h
Copy link
Contributor

primes2h commented Nov 29, 2018

Ci sarebbe anche questa eventualmente da includere #677, penso basti un cherry-picking quando verrà fatto il merge.

@sergiocorato
Copy link
Contributor

Nel modulo l10n_it_fatturapa_out vedo che c'è la dipendenza aggiuntiva alla libreria di python phonenumbers

@@ -216,7 +216,7 @@ def _setContattiTrasmittente(self, cr, uid, company, fatturapa, context=None):
return True

def checkSetupPhone(self, phone_number):
if '' in phone_number:
if '+' in phone_number:
Copy link
Contributor

Choose a reason for hiding this comment

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

Nel caso il numero telefonico non ci sia:
def checkSetupPhone(self, phone_number=False): if phone_number and '+' in phone_number:

oca-travis and others added 27 commits December 16, 2020 09:45
Updated by Update PO files to match POT (msgmerge) hook in Weblate.
@github-actions
Copy link

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 Oct 24, 2021
@github-actions github-actions bot closed this Nov 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stale PR/Issue without recent activity, it'll be soon closed automatically.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet