[14.0][MIG] l10n_it_website_portal_fatturapa #2520

Borruso · 2021-11-20T17:49:59Z

Migration l10n_it_website_portal_fatturapa from 12.0 to 14.0

Dipendenze l10n_it_website_portal_fiscalcode, l10n_it_website_portal_ipa

--
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

matteoopenf · 2022-02-04T17:22:55Z

Ti occorre review?

Borruso · 2022-02-04T17:50:47Z

Ti occorre review?

Sisi, se puoi fare review va benissimo

SimoRubi

Grazie della PR!
Ho provato il modulo in runboat e mi pare funzionare.
Durante la revisione del codice ho notato un paio di cose: l'unica secondo me da chiarire per poter andare avanti è il controllo rimosso.

SimoRubi · 2022-02-09T16:50:31Z

l10n_it_website_portal_fatturapa/static/src/js/l10n_it_website_portal_fatturapa.js

-        if (!$is_pa.length) {
-            return;
+            var compute_destinatario_fields_visibility = function () {
+                if ($is_pa[0].checked) {


Come mai questo if è stato invertito?

problemi con il pre-commit

SimoRubi · 2022-02-09T16:53:02Z

l10n_it_website_portal_fatturapa/static/src/js/l10n_it_website_portal_fatturapa.js

@@ -5,47 +5,46 @@ odoo.define("l10n_it_website_portal_fatturapa", function (require) {
    $(document).ready(function () {
        var details_div_selector = ".o_portal_details";
        var $details_div = $(details_div_selector);
-        if (!$details_div.length) {
-            return $.Deferred().reject("DOM doesn't contain " + details_div_selector);


Come mai questo controllo è stato rimosso?

ora non ricordo devo provare a ri-metterlo e fare dei test

il controllo scatta ogni volta che sei in una pagina dove il <div> del portale utente non c'è.
quindi questo controllo è totalmente inutile da questa versione in poi come alle v16.
ecco perché è stato rimosso.

matteoopenf · 2022-04-22T11:48:46Z

@Borruso per caso la riaggiorni a breve?

matteoopenf · 2022-09-06T07:40:45Z

@SimoRubi sono ok i fix che ha fatto @Borruso ?

matteoopenf

l'abbiamo provato su un sitoweb in fase di test e funziona

primes2h · 2022-11-08T15:08:29Z

/ocabot migration l10n_it_website_portal_fatturapa

matteoopenf · 2023-03-23T16:56:42Z

@SimoRubi qua i fix fatti sono ok? noi lo stiamo usando il modulo senza rilevare problemi

matteoopenf · 2023-04-14T15:49:28Z

@SimoRubi ci siamo qui?

sergiocorato · 2023-06-14T07:10:15Z

@SimoRubi ci siamo qui?

@SirTakobi

SirTakobi · 2023-06-14T08:10:45Z

@SimoRubi ci siamo qui?

@SirTakobi

Me lo segno tra le cose da guardare, ma non credo che riuscirò prossimamente.

La review di @SimoRubi rimarrà così perché l'utente non è più attivo

matteoopenf · 2023-07-12T10:23:17Z

@SirTakobi ci siamo?

SirTakobi · 2023-07-12T10:36:01Z

@SirTakobi ci siamo?

Da #2520 (comment) è cambiato solo che ora è tra le cose da guardare 😄

francesco-ooops · 2023-07-12T10:44:30Z

@Borruso riesci a far ricostruire il runboat?

Borruso · 2023-07-12T12:30:01Z

@Borruso riesci a far ricostruire il runboat?

fatto rebase vediamo se lo genera

matteoopenf · 2023-07-12T13:10:44Z

Ora c’è Matteo ***@***.*** Il giorno 12 lug 2023, alle ore 14:30, Giuseppe Borruso ***@***.***> ha scritto: @Borruso riesci a far ricostruire il runboat? fatto rebase vediamo se lo genera —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: ***@***.***> Openforce Srls Unipersonale | www.openforce.it | 02 21119472 | Via Tiziano,32 20145 Milano

francesco-ooops · 2023-07-12T13:52:03Z

l10n_it_website_portal_fatturapa/readme/ROADMAP.rst

@@ -0,0 +1 @@
+Fields `is_pa` and `ipa_code` are not shown in the frontend because Odoo core always supposes that frontend users are individual, and these fields are only meaningful for companies.


@Borruso questa sezione si può rimuovere, i due campi sono ora visibili nel portale se l'utente è legato ad un partner di tipo company

francesco-ooops

Funzionalmente non so dare un giudizio perchè se P.IVA o CF non sono valorizzati da backend l'utente non può attivare la fatturazione elettronica, dato che i due campi non sono editabili per qualche motivo https://recordit.co/m8x6pMTEFs

FrancescoBallerini · 2023-09-13T09:16:30Z

Funzionalmente non so dare un giudizio perchè se P.IVA o CF non sono valorizzati da backend l'utente non può attivare la fatturazione elettronica, dato che i due campi non sono editabili per qualche motivo https://recordit.co/m8x6pMTEFs

Stavo testando la PR, mi risulta che questo sia un comportamento slegato dalla PR e dovuto al fatto che l'utente abbia già almeno un ordine di vendita confermato/pagato (di cui è stato completato il checkout): quando l'utente esegue un'iscrizione da zero i due campi sono sempre impostabili, anche se in backend risultano non valorizzati.

francesco-ooops · 2023-09-13T09:53:26Z

@Borruso puoi far ripartire il runboat?

Borruso · 2023-09-13T14:14:01Z

@Borruso puoi far ripartire il runboat?

fatto

francesco-ooops

ok, funzionalmente si comporta come nella 12

FrancescoBallerini · 2023-09-25T10:14:15Z

l10n_it_website_portal_fatturapa/controllers/main.py

+        data["electronic_invoice_subjected"] = data.get(
+            "electronic_invoice_subjected", False
+        )
+        if data["electronic_invoice_subjected"]:


Cosa ne pensi di trasformare questo blocco in

if data["electronic_invoice_subjected"]: data["electronic_invoice_obliged_subject"] = True country_italy = request.env.ref('base.it', raise_if_not_found=False) country_id = data.get('country_id', False) if country_id and country_italy and int(country_id) != country_italy.id: data["codice_destinatario"] = "XXXXXXX" elif data.get('codice_destinatario') == "XXXXXXX": data["codice_destinatario"] = "0000000"

? per via di questo #2520 (comment)

@Borruso Grazie, un ultimo appunto è che si potrebbe importare il codice destinatario standard con "0000000" da qui https://github.com/OCA/l10n-italy/blob/14.0/l10n_it_fatturapa/models/partner.py#L7 , ma non so se ne vale la pena, decidete voi 👍

@Borruso Grazie, un ultimo appunto è che si potrebbe importare il codice destinatario standard con "0000000" da qui https://github.com/OCA/l10n-italy/blob/14.0/l10n_it_fatturapa/models/partner.py#L7 , ma non so se ne vale la pena, decidete voi 👍

puoi aprire una issue per tenerne traccia

FIX l10n_it_website_portal_fatturapa: Cannot read property 'checked' of undefined when not in /my/home page

Otherwise, when installed directly: File "odoo/addons/base/models/ir_model.py", line 1380, in xmlid_lookup raise ValueError('External ID not found in the system: %s' % xmlid) odoo.tools.convert.ParseError: "External ID not found in the system: website.assets_frontend" while parsing None:8, near <data inherit_id="website.assets_frontend" name="Website Portal Electronic Invoice"> <xpath expr="." position="inside"> <script type="text/javascript" src="/l10n_it_website_portal_fatturapa/static/src/js/l10n_it_website_portal_fatturapa.js"/> </xpath> </data>

FrancescoBallerini · 2023-10-05T09:47:52Z

l10n_it_website_portal_fatturapa/controllers/main.py

+        data["electronic_invoice_subjected"] = data.get(
+            "electronic_invoice_subjected", False
+        )
+        if data["electronic_invoice_subjected"]:


@Borruso Grazie, un ultimo appunto è che si potrebbe importare il codice destinatario standard con "0000000" da qui https://github.com/OCA/l10n-italy/blob/14.0/l10n_it_fatturapa/models/partner.py#L7 , ma non so se ne vale la pena, decidete voi 👍

sergiocorato · 2024-02-23T16:48:31Z

@Borruso riesci a squashare i commit non rilevanti come diceva @tafaRU ? credo siano questi:

…jected they are also obliged

Borruso · 2024-02-26T09:11:16Z

@Borruso riesci a squashare i commit non rilevanti come diceva @tafaRU ? credo siano questi:

fatto

matteoopenf · 2024-02-26T11:44:22Z

merge?

sergiocorato · 2024-03-01T08:59:19Z

/ocabot merge nobump

OCA-git-bot · 2024-03-01T08:59:27Z

This PR looks fantastic, let's merge it!
Prepared branch 14.0-ocabot-merge-pr-2520-by-sergiocorato-bump-nobump, awaiting test results.

OCA-git-bot · 2024-03-01T09:27:19Z

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

Borruso mentioned this pull request Nov 20, 2021

Migration to version 14.0 #1905

Open

76 tasks

Borruso force-pushed the 14.0-mig-l10n_it_website_portal_fatturapa branch from cd19afa to 8fdff13 Compare January 28, 2022 13:12

SimoRubi suggested changes Feb 9, 2022

View reviewed changes

Borruso force-pushed the 14.0-mig-l10n_it_website_portal_fatturapa branch from 8fdff13 to 0ce49df Compare April 26, 2022 08:33

Borruso force-pushed the 14.0-mig-l10n_it_website_portal_fatturapa branch from 0ce49df to 433e582 Compare September 6, 2022 08:54

matteoopenf approved these changes Sep 13, 2022

View reviewed changes

lorenzoallegrucci approved these changes Sep 13, 2022

View reviewed changes

OCA-git-bot added this to the 14.0 milestone Nov 8, 2022

Borruso force-pushed the 14.0-mig-l10n_it_website_portal_fatturapa branch from 433e582 to e24ac96 Compare July 12, 2023 12:28

francesco-ooops reviewed Jul 12, 2023

View reviewed changes

Borruso force-pushed the 14.0-mig-l10n_it_website_portal_fatturapa branch from e24ac96 to ef03629 Compare September 13, 2023 14:13

francesco-ooops approved these changes Sep 13, 2023

View reviewed changes

FrancescoBallerini suggested changes Sep 25, 2023

View reviewed changes

monen17 and others added 4 commits October 5, 2023 11:05

[ADD] l10n_it_website_portal_fatturapa

418a959

porting l10n_it_website_portal_fatturapa to 12

c9d7a9b

FIX l10n_it_website_portal_fatturapa: Cannot read property 'checked' of undefined when not in /my/home page

[IMP] l10n_it_website_portal_fatturapa: Add PA check, IPA and EORI code

d860db5

Borruso force-pushed the 14.0-mig-l10n_it_website_portal_fatturapa branch from dffcafb to 13a03bc Compare October 5, 2023 09:13

Borruso requested a review from FrancescoBallerini October 5, 2023 09:24

FrancescoBallerini approved these changes Oct 5, 2023

View reviewed changes

SirAionTech mentioned this pull request Oct 5, 2023

[16.0] [MIG] l10n_it_website_portal_fatturapa #3406

Merged

Borruso force-pushed the 14.0-mig-l10n_it_website_portal_fatturapa branch from 13a03bc to c2e234f Compare October 5, 2023 16:37

FrancescoBallerini mentioned this pull request Oct 5, 2023

l10n_it_website_portal_fatturapa: implementazione gestione utente dal portale #3644

Closed

2 tasks

Borruso requested a review from SimoRubi October 8, 2023 16:46

francesco-ooops added 14.0 migration labels Dec 20, 2023

sergiocorato approved these changes Feb 23, 2024

View reviewed changes

SimoRubi and others added 4 commits February 26, 2024 10:10

[FIX] l10n_it_website_portal_fatturapa: Fix electronic invoice label

3d0fbcb

IMP l10n_it_website_portal_fatturapa: when customers set they are sub…

030f28e

…jected they are also obliged

[IMP] l10n_it_website_portal_fatturapa: black, isort, prettier

9adc0a1

[MIG] l10n_it_website_portal_fatturapa: Migration to 14.0

5d7b489

Borruso force-pushed the 14.0-mig-l10n_it_website_portal_fatturapa branch from c2e234f to 5d7b489 Compare February 26, 2024 09:10

OCA-git-bot added the bot is merging ⏳ label Mar 1, 2024

OCA-git-bot merged commit 9d2550c into OCA:14.0 Mar 1, 2024
6 of 7 checks passed

OCA-git-bot added merged 🎉 and removed bot is merging ⏳ labels Mar 1, 2024

		@@ -0,0 +1 @@
		Fields `is_pa` and `ipa_code` are not shown in the frontend because Odoo core always supposes that frontend users are individual, and these fields are only meaningful for companies.

[14.0][MIG] l10n_it_website_portal_fatturapa #2520

[14.0][MIG] l10n_it_website_portal_fatturapa #2520

Conversation

Borruso commented Nov 20, 2021 • edited Loading

matteoopenf commented Feb 4, 2022

Borruso commented Feb 4, 2022

SimoRubi left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

matteoopenf commented Apr 22, 2022

matteoopenf commented Sep 6, 2022

matteoopenf left a comment

Choose a reason for hiding this comment

primes2h commented Nov 8, 2022

matteoopenf commented Mar 23, 2023

matteoopenf commented Apr 14, 2023

sergiocorato commented Jun 14, 2023

SirTakobi commented Jun 14, 2023

matteoopenf commented Jul 12, 2023

SirTakobi commented Jul 12, 2023

francesco-ooops commented Jul 12, 2023

Borruso commented Jul 12, 2023

matteoopenf commented Jul 12, 2023 via email

francesco-ooops Jul 12, 2023 • edited Loading

Choose a reason for hiding this comment

francesco-ooops left a comment

Choose a reason for hiding this comment

FrancescoBallerini commented Sep 13, 2023

francesco-ooops commented Sep 13, 2023

Borruso commented Sep 13, 2023

francesco-ooops left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

FrancescoBallerini Oct 5, 2023 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

FrancescoBallerini Oct 5, 2023 • edited Loading

Choose a reason for hiding this comment

sergiocorato commented Feb 23, 2024

Borruso commented Feb 26, 2024 • edited Loading

matteoopenf commented Feb 26, 2024

sergiocorato commented Mar 1, 2024

OCA-git-bot commented Mar 1, 2024

OCA-git-bot commented Mar 1, 2024

Borruso commented Nov 20, 2021 •

edited

Loading

francesco-ooops Jul 12, 2023 •

edited

Loading

FrancescoBallerini Oct 5, 2023 •

edited

Loading

FrancescoBallerini Oct 5, 2023 •

edited

Loading

Borruso commented Feb 26, 2024 •

edited

Loading