[14.0][MIG] l10n_it_fatturapa_out_triple_discount by GSLabIt · Pull Request #2760 · OCA/l10n-italy

GSLabIt · 2022-04-13T15:50:05Z

#1905

TheMule71 · 2022-04-15T07:31:11Z

Prova a fare un rebase sulla mia #2753 e poi un push -f

francesco-ooops

LGTM

GSLabIt · 2022-04-20T13:58:55Z

Prova a fare un rebase sulla mia #2753 e poi un push -f

La pr indicata è chiusa. Ho fatto un rebase su 14.0, ma non ha risolto il problema. Qualche suggerimento?

TheMule71 · 2022-04-20T20:22:06Z

        </template>

        <template id="account_invoice_line_it_sconto_maggiorazione">
-            <t t-set="importo" t-value="get_importo(line)" />


Non riesco a capire a che serve la modifica.

Serve a non riscrivere la funzione per il calcolo dello sconto, solo per questo.

TheMule71 · 2022-04-20T20:24:54Z

            out.update(out_computed)
            return out

-        def get_importo(line):


Trovo più versatile passare l'intera line piuttosto che i due valori qui. Un sottomodulo potrebbe voler fare l'override dell'importo leggendo valori presi da line. Se passi solo price_unit e discount, altri moduli non ci possono fare molto.

A me sembra più utile non duplicare il codice per il calcolo dello sconto.

Beh prima c'era funzione get_importo() che riceveva un parametro, line, adesso riceve due parametri, price_unit e discount, entrambi però presi da line.
È cambiata la signature della funzione, e rende la funzione meno versatile.

Forse sarebbe meglio passare un parametro aggiuntivo, opzionale, che permetta di selezionare quale sconto applicare, mantenendo la compatibilità con la base e altri (eventuali) moduli. Tipo:

def get_importo(line, discount_field="discount"): str_number = str(line[discount_field])

che permette di mantenere la compatibilità e comunque scegliere lo sconto da applicare, in modo del tutto generale. Un domani un altro modulo potrebbe voler fare dei conti sull'importo usando campi dalla line, o introdurre in altro campo sconto da usare come base tipo discount4 (faccio per dire, non che abbia senso).

Chiaro ma resto della mia idea; mi da l'impressione di un UCAS. Detto ciò, sentiti libero di modificare la pr.

francesco-ooops · 2022-04-28T07:38:58Z

@OCA/local-italy-maintainers come possiamo procedere con questa?

francesco-ooops · 2022-05-13T14:44:59Z

@OCA/local-italy-maintainers potete dare un'occhio anche a questo porting? grazie!

francesco-ooops · 2022-05-23T13:47:06Z

@TheMule71 puoi suggerire come implementeresti la modifica richiesta? che sennò restiamo nell'impasse e non possiamo gestire lo sconto triplo in fattura. Grazie!

francesco-ooops · 2022-06-13T14:33:21Z

@SirTakobi avresti qualche indicazione su come procedere con questa migrazione?

SirTakobi · 2022-06-15T08:54:24Z

@SirTakobi avresti qualche indicazione su come procedere con questa migrazione?

Me l'ero già segnata grazie, quando ho tempo ci guardo

SirTakobi

Puoi correggere i test?
Suggerisco anche GSLabIt#2 per risolvere la discussione partita da #2760 (comment).
cc @TheMule71

SirTakobi · 2022-07-13T08:38:01Z

 # TODO https://github.com/OCA/l10n-italy/issues/1588
-#from . import test_fatturapa_triple_discount
+# from . import test_fatturapa_triple_discount


Puoi verificare se questo problema continua a verificarsi ed eventualmente ripristinare i test?

@SirTakobi i test continuano a fallire in l10n_it_fatturapa_in e l10n_it_fatturapa_in_purchase, hai qualche suggerimento?

@SirTakobi i test continuano a fallire in l10n_it_fatturapa_in e l10n_it_fatturapa_in_purchase, hai qualche suggerimento?

Nel commento indicavo i test di l10n_it_fatturapa_out_triple_discount, non quelli di altri moduli.

Comunque, per quanto riguarda i test di l10n_it_fatturapa_in e l10n_it_fatturapa_in_purchase: se lo riproduci in locale, andrei in debug.
Puoi anche provare se OCA/account-invoicing#876 risolve.

primes2h · 2022-11-08T15:04:15Z

/ocabot migration l10n_it_fatturapa_out_triple_discount

matteoopenf · 2022-12-30T11:34:54Z

per caso ci sono novità su questa pr?

francesco-ooops · 2022-12-30T11:53:36Z

per caso ci sono novità su questa pr?

per il momento no, se volete portarla avanti siamo molto contenti :)

odooNextev · 2023-04-11T15:17:20Z

Utilizzando la versione 5.12.0 di isort nel file pre-commit non da problemi nei test.

…API and to new file names

it fixes: - removes xs:dateTime if bogus and not mandatory i.e. 0001-01-01T00:00:00.000+02:00 where python raises OverflowError - removes timezone from xs:date to make pyxb able to compare with 1-1-1970, it also removes the need of patching pyxb - removes space only strings if not mandatory, else replace with a dash breaking change: modules needs to import binding.fatturapa instead of bindings.fatturapa_v_1_2, this would be asl useful for new specs

…e now rounded to the nearest even result instead of away from zero". See https://docs.python.org/3/whatsnew/3.0.html#builtins Examples: >>> '%.2f' % 75.845 '75.84' >>> '%.2f' % 75.855 '75.86' >>> round(2.5) 2 >>> round(3.5) 4

…scount: modifica del type di Importo in ScontoMaggiorazioneType (OCA#1875) * [IMP] l10n_it_fatturapa_out: change precision_rounding of ScontoMaggiorazioneType according to e-invoicing specs 1.6 * [IMP] l10n_it_fatturapa_out_triple_discount: change precision_rounding of ScontoMaggiorazioneType according to e-invoicing specs 1.6

… discard

GSLabIt · 2023-04-11T17:16:02Z

Utilizzando la versione 5.12.0 di isort nel file pre-commit non da problemi nei test.

il pre-commit non ha dato errori, ho cmq fatto un rebase.

vediamo come vanno

francesco-ooops · 2023-04-12T09:16:51Z

@odooNextev se volete riprendere il lavoro in una nuova PR, siete benvenuti :)

tafaRU · 2023-04-14T08:06:12Z

superseeded by #3269

GSLabIt force-pushed the 14.0-mig-l10n_it_fatturapa_out_triple_discount branch from 85306ed to 6f28831 Compare April 13, 2022 15:56

GSLabIt mentioned this pull request Apr 13, 2022

Migration to version 14.0 #1905

Open

76 tasks

matteoopenf approved these changes Apr 14, 2022

View reviewed changes

francesco-ooops approved these changes Apr 19, 2022

View reviewed changes

OCA-git-bot added the approved label Apr 19, 2022

GSLabIt force-pushed the 14.0-mig-l10n_it_fatturapa_out_triple_discount branch from 6f28831 to 7eb8811 Compare April 20, 2022 13:44

TheMule71 reviewed Apr 20, 2022

View reviewed changes

SirTakobi mentioned this pull request Jul 13, 2022

Minimizzare le modifiche al metodo get_importo. GSLabIt/l10n-italy#2

Merged

SirTakobi suggested changes Jul 13, 2022

View reviewed changes

OCA-git-bot removed the approved label Jul 13, 2022

GSLabIt force-pushed the 14.0-mig-l10n_it_fatturapa_out_triple_discount branch 2 times, most recently from 1fa35cc to c372328 Compare July 14, 2022 09:49

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

GSLabIt force-pushed the 14.0-mig-l10n_it_fatturapa_out_triple_discount branch from c372328 to 513a474 Compare January 10, 2023 08:05

SimoRubi and others added 4 commits April 11, 2023 19:08

[ADD] l10n_it_fatturapa_out_triple_discount

01b67c2

[UPD] Update l10n_it_fatturapa_out_triple_discount.pot

3df7fca

FIX l10n_it_fatturapa_out_triple_discount adapt to new set_sequences …

4017fce

…API and to new file names

[ADD] icon.png

9c6c605

sherpya and others added 18 commits April 11, 2023 19:08

[UPD] README.rst

264f9b7

[MIG] 10n_it_fatturapa_out_triple_discount: Migration to 12.0

5b76fae

[UPD] Update l10n_it_fatturapa_out_triple_discount.pot

3b7b6c9

[UPD] README.rst

2278bbc

l10n_it_fatturapa_out_triple_discount 12.0.1.0.1

c640350

l10n_it_fatturapa_out_triple_discount: disable tests because unstable

9c9bc3a

l10n_it_fatturapa_out_triple_discount 12.0.1.0.2

4e40600

merge PR 1264 1717 1728 1737 1739

686943b

FIX license LGPL-3 conflicts

311e046

l10n_it_fatturapa_out_triple_discount 12.0.1.0.4

84b0ad1

l10n_it_fatturapa_out_triple_discount 12.0.2.0.0

641d54b

[FIX][l10n_it_fatturapa_out_triple_discount] change rounding to avoid…

5cf5ef3

… discard

l10n_it_fatturapa_out_triple_discount 12.0.2.0.1

dbad815

[REF] l10n_it_fatturapa_out: Extract discounted price computation

c87162f

[MIG] l10n_it_fatturapa_out_triple_discount: Migration to 14.0

0c28ad6

GSLabIt force-pushed the 14.0-mig-l10n_it_fatturapa_out_triple_discount branch from 513a474 to 0c28ad6 Compare April 11, 2023 17:09

tafaRU closed this Apr 14, 2023

Uh oh!

Conversation

GSLabIt commented Apr 13, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

TheMule71 commented Apr 15, 2022

Uh oh!

francesco-ooops left a comment

Choose a reason for hiding this comment

Uh oh!

GSLabIt commented Apr 20, 2022

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!

francesco-ooops commented Apr 28, 2022

Uh oh!

francesco-ooops commented May 13, 2022

Uh oh!

francesco-ooops commented May 23, 2022

Uh oh!

francesco-ooops commented Jun 13, 2022

Uh oh!

SirTakobi commented Jun 15, 2022

Uh oh!

SirTakobi left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

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!

primes2h commented Nov 8, 2022

Uh oh!

matteoopenf commented Dec 30, 2022

Uh oh!

francesco-ooops commented Dec 30, 2022

Uh oh!

odooNextev commented Apr 11, 2023

Uh oh!

GSLabIt commented Apr 11, 2023

Uh oh!

francesco-ooops commented Apr 12, 2023

Uh oh!

tafaRU commented Apr 14, 2023

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

13 participants

GSLabIt commented Apr 13, 2022 •

edited

Loading