[14.0][MIG] l10n_it_sdi_channel: Migration to 14.0 by Borruso · Pull Request #2157 · OCA/l10n-italy

Borruso · 2021-03-05T17:15:25Z

Porting del modulo l10n_it_sdi_channel dalla versione 12.0 alla 14.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

andreampiovesana

go

GSLabIt

Non si potrebbe fare un refactoring e "disaccoppiare" tutto ciò che riguarda la pec e spostarlo su l10n_it_fatturapa_pec?

GSLabIt · 2021-03-06T07:55:03Z

+    email_from_for_fatturaPA = fields.Char(
+        string="Sender Email Address",
+        related="sdi_channel_id.pec_server_id.email_from_for_fatturaPA",
+        readonly=True,
+    )
+    email_exchange_system = fields.Char(
+        string="Exchange System Email Address",
+        related="sdi_channel_id.email_exchange_system",
+        readonly=True,
+    )


Non si potrebbero spostare nel modulo fatturapa_pec?

Dato che sulla v12 è ancora così, potresti creare una issue con la modifica da apportare in entrambe le versioni

GSLabIt · 2021-03-06T07:55:18Z

+    email_from_for_fatturaPA = fields.Char(
+        string="Sender Email Address",
+        related="sdi_channel_id.pec_server_id.email_from_for_fatturaPA",
+        readonly=True,
+    )
+    email_exchange_system = fields.Char(
+        string="Exchange System Email Address",
+        related="sdi_channel_id.email_exchange_system",
+        readonly=True,
+    )


tafaRU · 2021-04-14T12:51:53Z

@Borruso vedi #1959 (comment)

Borruso · 2021-04-16T08:05:14Z

@Borruso vedi #1959 (comment)

Fatto ma non compare Travis

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: l10n-italy-12.0/l10n-italy-12.0-l10n_it_sdi_channel Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-12-0/l10n-italy-12-0-l10n_it_sdi_channel/

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

…plicable to tree view

…g SMTP server: the e-invoice PEC server must not be used

OCA-git-bot · 2021-10-15T13:08:53Z

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

TheMule71

LGTM
Unica cosa di cui non sono sicuro - chiedo a chi fa traduzioni - il file it.po va incluso nella migrazione o va lasciato fuori e rigenerato via weblate?

TheMule71 · 2021-10-25T10:36:18Z

+                <header>
+                </header>


L'header vuoto fa qualcosa?

in effetti è la stessa form che abbiamo per la 12.0
per me possiamo eliminarle dato che non dovrebbero servire a nulla

Magari è li per estendere la form ed aggiungere eventuali buttons?

Magari è li per estendere la form ed aggiungere eventuali buttons?

vero ma si potrebbe fare un <xpath expr="//form/sheet" position="before">

Vero anche questo. Magari su 14.0 si può rimuovere se non vi sono altri moduli che lo estendono basandosi su <header>.

a questo punto potremmo inserite anche questa modifica nella issue del refactoring 12.0/14.0 che hai scritto e valutare di eliminarlo?

Aggiungi pure

tafaRU · 2021-11-02T07:56:27Z

@TheMule71

il file it.po va incluso nella migrazione o va lasciato fuori e rigenerato via weblate?

se ne era parlato nel canale Discord #traduzioni di Odoo Italia giungendo alla conclusione che non sono da rimuovere altrimenti verrebbero rigenerati vuoti perdendo così di fatto tutte le vecchie traduzioni.

TheMule71 · 2021-11-12T08:50:44Z

/ocabot merge nobump

OCA-git-bot · 2021-11-12T08:50:55Z

On my way to merge this fine PR!
Prepared branch 14.0-ocabot-merge-pr-2157-by-TheMule71-bump-nobump, awaiting test results.

OCA-git-bot · 2021-11-12T09:14:08Z

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

Borruso changed the title ~~Porting del modulo l10n_it_sdi_channel dalla versione 12.0 alla 14.0~~ [14.0][MIG] l10n_it_sdi_channel: Migration to 14.0 Mar 5, 2021

Borruso mentioned this pull request Mar 5, 2021

Migration to version 14.0 #1905

Open

76 tasks

andreampiovesana approved these changes Mar 5, 2021

View reviewed changes

GSLabIt reviewed Mar 6, 2021

View reviewed changes

TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Mar 26, 2021

add OCA#2157

c1bfb8a

TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Apr 15, 2021

add OCA#2157

bc34151

Borruso force-pushed the 14.0-mig-l10n_it_sdi_channel branch from d5cc81f to d1ebec8 Compare April 16, 2021 07:43

TheMule71 reviewed May 5, 2021

View reviewed changes

Comment thread l10n_it_sdi_channel/models/sdi.py Outdated

Borruso force-pushed the 14.0-mig-l10n_it_sdi_channel branch from d1ebec8 to a33e068 Compare May 6, 2021 05:53

Borruso requested a review from TheMule71 May 6, 2021 05:53

Borruso force-pushed the 14.0-mig-l10n_it_sdi_channel branch 3 times, most recently from 57d2216 to 4f630f3 Compare May 6, 2021 06:14

TheMule71 removed their request for review May 7, 2021 08:50

Borruso force-pushed the 14.0-mig-l10n_it_sdi_channel branch from 4f630f3 to 8fc01da Compare June 4, 2021 14:14

TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jun 6, 2021

add OCA#2157

b8715d7

TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jun 18, 2021

add OCA#2157

9bb72f3

TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jun 18, 2021

add OCA#2157

8de3aa8

jado95 approved these changes Jul 1, 2021

View reviewed changes

OCA-git-bot added the approved label Jul 1, 2021

TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 2, 2021

add OCA#2157

2636901

TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 2, 2021

add OCA#2157

78b7700

TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 9, 2021

add OCA#2157

4fd8e72

TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 9, 2021

add OCA#2157

987ebb0

TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 9, 2021

add OCA#2157

f1bedd7

TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 9, 2021

add OCA#2157

a090076

TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 17, 2021

add OCA#2157

92e032e

TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 20, 2021

add OCA#2157

1b2e021

OCA-git-bot and others added 13 commits October 15, 2021 09:58

l10n_it_sdi_channel 12.0.1.2.0

5af8bdc

Update translation files

80c037c

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: l10n-italy-12.0/l10n-italy-12.0-l10n_it_sdi_channel Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-12-0/l10n-italy-12-0-l10n_it_sdi_channel/

Translated using Weblate (Italian)

e4df45c

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

Translated using Weblate (Italian)

37f00fa

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

IMP l10n_it_sdi_channel hiding "web server" fields ad attrs is not ap…

97154e3

…plicable to tree view

l10n_it_sdi_channel 12.0.1.3.0

3a1e203

[12.0][IMP] l10n_it_sdi_channel, add form header

306accd

merge PR 1264 1717 1728 1737 1739

1a2c62d

l10n_it_sdi_channel 12.0.1.3.2

b37c7a1

IMP l10n_it_sdi_channel when sending normal emails without explicitin…

14b1168

…g SMTP server: the e-invoice PEC server must not be used

l10n_it_sdi_channel 12.0.1.3.3

e44e948

[IMP] l10n_it_sdi_channel: black, isort, prettier

2c24436

[MIG] l10n_it_sdi_channel: Migration to 14.0

d630333

Borruso force-pushed the 14.0-mig-l10n_it_sdi_channel branch from b7fc107 to d630333 Compare October 15, 2021 09:12

OCA-git-bot added the ready to merge label Oct 15, 2021

TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Oct 15, 2021

add OCA#2157

8e4f575

TheMule71 approved these changes Oct 25, 2021

View reviewed changes

OCA-git-bot added the bot is merging ⏳ label Nov 12, 2021

OCA-git-bot merged commit 02bdc28 into OCA:14.0 Nov 12, 2021

OCA-git-bot added merged 🎉 and removed bot is merging ⏳ labels Nov 12, 2021

TheMule71 added a commit to TheMule71/l10n-italy that referenced this pull request Jan 21, 2022

add OCA#2157

a57e580

TheMule71 added a commit to TheMule71/l10n-italy that referenced this pull request Jan 21, 2022

add OCA#2157

5bede9e

TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jun 29, 2022

add OCA#2157

d7892c1

francesco-ooops added the migration label Jan 10, 2024

Uh oh!

Conversation

Borruso commented Mar 5, 2021

Uh oh!

andreampiovesana left a comment

Choose a reason for hiding this comment

Uh oh!

GSLabIt left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

tafaRU commented Apr 14, 2021

Uh oh!

Borruso commented Apr 16, 2021

Uh oh!

Uh oh!

OCA-git-bot commented Oct 15, 2021

Uh oh!

TheMule71 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

tafaRU commented Nov 2, 2021

Uh oh!

TheMule71 commented Nov 12, 2021

Uh oh!

OCA-git-bot commented Nov 12, 2021

Uh oh!

OCA-git-bot commented Nov 12, 2021

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

14 participants