Skip to content

[14.0][ADD] new module l10n_it_fatturapa_out_sp#2154

Merged
OCA-git-bot merged 1 commit into
OCA:14.0from
TheMule71:14.0-l10n_fatturapa_out_sp
Sep 13, 2021
Merged

[14.0][ADD] new module l10n_it_fatturapa_out_sp#2154
OCA-git-bot merged 1 commit into
OCA:14.0from
TheMule71:14.0-l10n_fatturapa_out_sp

Conversation

@TheMule71

Copy link
Copy Markdown
Contributor

Integration between l10n_fatturapa_out and l10n_split_payment

a seguito di 75c6fff la funzionalità è stata separata

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

@TheMule71 TheMule71 mentioned this pull request Mar 5, 2021
76 tasks
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Mar 5, 2021
@eLBati

eLBati commented Mar 5, 2021

Copy link
Copy Markdown
Member

Il commit 75c6fff in che PR/branch sta?
Questa separazione prevedete di farla all'interno del processo di porting alla 14?

@TheMule71 TheMule71 force-pushed the 14.0-l10n_fatturapa_out_sp branch from ca3888c to bc09699 Compare March 5, 2021 18:28
@TheMule71

Copy link
Copy Markdown
Contributor Author

Il commit 75c6fff in che PR/branch sta?
Questa separazione prevedete di farla all'interno del processo di porting alla 14?

Si, per favorire l'eventuale merge di fatturapa_out, se per caso al momento split_payment non fosse pronto.

Di fatto la dipendenza che c'era era in quelle quattro righe di template: 75c6fff#diff-11dc48afbaae626667c7075d54bb69cb88a77b2a679a3eb4affdb6eeef377c5cL539 e basta.

Alla fine il commit definitivo dovrebbe essere questo: 4620db8
il 75c6fff probabilmente nel corso della giornata è finito detacchato a causa di un push -f.

@TheMule71 TheMule71 force-pushed the 14.0-l10n_fatturapa_out_sp branch from bc09699 to a97b67e Compare March 12, 2021 18:16
@TheMule71 TheMule71 force-pushed the 14.0-l10n_fatturapa_out_sp branch from a97b67e to 94a301f Compare April 15, 2021 15:08
@tafaRU

tafaRU commented Apr 23, 2021

Copy link
Copy Markdown
Member

@TheMule71 vedi #1959 (comment)

@TheMule71 TheMule71 force-pushed the 14.0-l10n_fatturapa_out_sp branch from bce3312 to 8f4339c Compare May 14, 2021 15:07
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jun 6, 2021
@TheMule71

Copy link
Copy Markdown
Contributor Author

@TheMule71 TheMule71 force-pushed the 14.0-l10n_fatturapa_out_sp branch from d4db9ce to f596ca2 Compare June 9, 2021 09:17
@TheMule71 TheMule71 force-pushed the 14.0-l10n_fatturapa_out_sp branch 2 times, most recently from b1eb3d6 to 3f81732 Compare June 18, 2021 12:52
@TheMule71

Copy link
Copy Markdown
Contributor Author

Il problema del test sembra legato al modo in cui l10n_it_split_payment calcola i pagamenti:
AssertionError: b'<ns[2654 chars]nto>14.00</ImportoPagamento></DettaglioPagamen[65 chars]ica>' != b'<ns[2654 chars]nto>17.08</ImportoPagamento></DettaglioPagamen[65 chars]ica>'
14.00 è il valore atteso nel file, 17.08 è quello che si trova in fattura creata nel test, ma il test è copiato dalla 12.0.

@TheMule71

Copy link
Copy Markdown
Contributor Author

Vd. #2336, dovrebbe sistemare il problema dei test.

@TheMule71 TheMule71 marked this pull request as ready for review June 18, 2021 13:19
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jun 18, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jun 18, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 2, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 2, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 9, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 9, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 9, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 9, 2021
@TheMule71 TheMule71 force-pushed the 14.0-l10n_fatturapa_out_sp branch from 3f81732 to a9985ea Compare July 17, 2021 13:48
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 17, 2021
Comment thread l10n_it_fatturapa_out_sp/tests/__init__.py Outdated
@TheMule71 TheMule71 force-pushed the 14.0-l10n_fatturapa_out_sp branch from 53584a6 to 217792f Compare September 3, 2021 07:52
@TheMule71 TheMule71 requested a review from tafaRU September 3, 2021 07:52
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Sep 3, 2021
Comment thread l10n_it_fatturapa_out_sp/__manifest__.py Outdated
Integration between l10n_fatturapa_out and l10n_split_payment
@TheMule71 TheMule71 force-pushed the 14.0-l10n_fatturapa_out_sp branch from 217792f to b84e7c5 Compare September 10, 2021 14:29
@tafaRU

tafaRU commented Sep 13, 2021

Copy link
Copy Markdown
Member

/ocabot merge nobump

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 14.0-ocabot-merge-pr-2154-by-tafaRU-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit d4be97b into OCA:14.0 Sep 13, 2021
@OCA-git-bot

Copy link
Copy Markdown
Contributor

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

@TheMule71 TheMule71 deleted the 14.0-l10n_fatturapa_out_sp branch September 17, 2021 13:45
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Sep 17, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Sep 17, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Oct 1, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Oct 1, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Oct 8, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Oct 8, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Oct 15, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Oct 15, 2021
TheMule71 added a commit to TheMule71/l10n-italy that referenced this pull request Jan 21, 2022
TheMule71 added a commit to TheMule71/l10n-italy that referenced this pull request Jan 21, 2022
TheMule71 added a commit to TheMule71/l10n-italy that referenced this pull request Jan 21, 2022
TheMule71 added a commit to TheMule71/l10n-italy that referenced this pull request Jan 21, 2022
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jun 29, 2022
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jun 29, 2022
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.

4 participants