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

[16.0][MIG] l10n_it_fatturapa_pec #3190

Merged
merged 65 commits into from
Sep 7, 2023

Conversation

Borruso
Copy link
Contributor

@Borruso Borruso commented Feb 24, 2023

Migration l10n_it_fatturapa_pec from 14.0 to 16.0

--
Confermo di aver firmato il CLA https://odoo-community.org/page/cla e di aver letto le linee guida su https://odoo-community.org/page/contributing

@Borruso Borruso mentioned this pull request Feb 24, 2023
79 tasks
@tafaRU
Copy link
Member

tafaRU commented Feb 24, 2023

/ocabot migration l10n_it_fatturapa_pec

@OCA-git-bot OCA-git-bot added this to the 16.0 milestone Feb 24, 2023
@Borruso Borruso force-pushed the 16.0-mig-l10n_it_fatturapa_pec branch from fc2c05e to bfd7622 Compare March 10, 2023 09:56
@Borruso Borruso force-pushed the 16.0-mig-l10n_it_fatturapa_pec branch from bfd7622 to ab9d8c1 Compare July 6, 2023 16:00
@Borruso Borruso force-pushed the 16.0-mig-l10n_it_fatturapa_pec branch 4 times, most recently from ccdc0cc to e92e153 Compare July 21, 2023 09:11
eLBati and others added 20 commits July 21, 2023 12:06
FIX
Mail delivery failed via SMTP server 'smtps.pec.aruba.it'.
SMTPRecipientsRefused: {'[email protected]': (553, '5.7.1 <[email protected]>: Sender address rejected: not owned by user [email protected]')}
Traceback (most recent call last):
  File "/home/elbati/workspace/odoo/instances/demo10-community/parts/odoo/addons/mail/models/mail_mail.py", line 278, in send
    res = IrMailServer.send_email(msg, mail_server_id=mail.mail_server_id.id)
  File "/home/elbati/workspace/odoo/instances/demo10-community/parts/odoo/odoo/addons/base/ir/ir_mail_server.py", line 467, in send_email
    raise MailDeliveryException(_("Mail Delivery Failed"), msg)
MailDeliveryException: (u'Mail Delivery Failed', u"Mail delivery failed via SMTP server 'smtps.pec.aruba.it'.\nSMTPRecipientsRefused: {'[email protected]': (553, '5.7.1 <[email protected]>: Sender address rejected: not owned by user [email protected]')}")

ADD l10n_it_fatturapa_out: tracking attachment changes

IMP mail send: raise exception in case

[IMP] default is_fatturapa_pec and split class

REF  mergin l10n_it_fatturapa_in_pec and l10n_it_fatturapa_out_pec in l10n_it_fatturapa_pec

REF  mergin l10n_it_fatturapa_in_pec and l10n_it_fatturapa_out_pec in l10n_it_fatturapa_pec

FIX default_is_fatturapa_pec
[FIX] hide domain
exceptions sending email
Avoid to delete sent files
Check duplicated files
Send several files
Reset to ready button
Sending and delivery date
Handling ACCETTAZIONE and CONSEGNA
IMP 'link to existing invoice' wizard
FIX invoice views within attachment view
IMP attachments views
IMP l10n_it_fatturapa_out export: allow multiple export , group by partner
ALLOW to use newline char in invoice line description
check sequence without date ranges
Extract correcttle messages from NS
ADD mechanism to check SDI PEC address after first sending
ADD check valid PEC incoming server

IMP attachment out filter
…A#682)

While sending, check for valid fetchmail server
FIX typo 'yo'
[FIX] l10n_it_fatturapa_pec translation source strings

[IMP] l10n_it_sdi_channel revamp README and manifest (OCA#692)

[FIX] l10n_it_sdi_channel FatturaPA/translation source strings

[IMP] l10n_it_sdi_channel set ITA name and summary

[IMP] fatturapa modules, set ITA name and summary (OCA#694)

[FIX] l10n_it_fatturapa_in typos in source translation strings
…ut using mail.catchall parameters anyway, using always 'email_from_for_fatturaPA' (OCA#695)

FIX missing body_html
Currently translated at 100.0% (42 of 42 strings)

Translation: l10n-italy-12.0/l10n-italy-12.0-l10n_it_fatturapa_pec
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-12-0/l10n-italy-12-0-l10n_it_fatturapa_pec/it/
Currently translated at 100.0% (42 of 42 strings)

Translation: l10n-italy-12.0/l10n-italy-12.0-l10n_it_fatturapa_pec
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-12-0/l10n-italy-12-0-l10n_it_fatturapa_pec/it/
…otified when a non e-invoice PEC is received by fetchmail (OCA#733)
…mission error (OCA#743)

Sorry, you are not allowed to access this document. Only users with the following access level are currently allowed to do that:
- Administration/Settings

(Document model: fetchmail.server)
…ndividual without PEC or codice destinatario, so it sould not be an error. (OCA#813)

Added tests

Also "avvenuta trasmissione della fattura con impossibilità di recapito" notification must be handled and status tracked
Currently translated at 100.0% (47 of 47 strings)

Translation: l10n-italy-12.0/l10n-italy-12.0-l10n_it_fatturapa_pec
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-12-0/l10n-italy-12-0-l10n_it_fatturapa_pec/it/
…of temporary errors. (OCA#920)

Retry N (configurable) times before resetting fetchmail server.
Notify contacts about reset
Copy link
Contributor

@TheMule71 TheMule71 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link

@marcelofrare marcelofrare left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@eLBati eLBati left a comment

Choose a reason for hiding this comment

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

Sulla 14 sono state rilasciate queste modifiche #3118 che quindi sarebbero da includere in questo modulo, verificando che anche gli altri moduli coinvolti siano aggiornati

@eLBati
Copy link
Member

eLBati commented Sep 7, 2023

La PR #3118 modifica altri moduli che sono già merged su 14.0, quindi conviene fare il merge di questa e poi portare la #3118 alla 16

@eLBati
Copy link
Member

eLBati commented Sep 7, 2023

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 16.0-ocabot-merge-pr-3190-by-eLBati-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 01fe590 into OCA:16.0 Sep 7, 2023
5 of 6 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 1866889. Thanks a lot for contributing to OCA. ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.