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.

Sign up for GitHub

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

Jump to bottom

[14.0] [IMP] l10n_it_fatturapa_out: EUR values from journal #3299

Closed
wants to merge 3 commits into from
Closed

[14.0] [IMP] l10n_it_fatturapa_out: EUR values from journal #3299

wants to merge 3 commits into from

Conversation

Borruso
Copy link
Contributor

@Borruso Borruso commented May 10, 2023

Vedi #3295

--
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
Copy link
Contributor

Sei partito da questa? #3284

@Borruso
Copy link
Contributor Author

Borruso commented May 10, 2023

Sei partito da questa? #3284

sisi uguale come fatto per la 16

@Borruso Borruso force-pushed the 14.0-imp-l10n_it_fatturapa_out-eur-from-journal branch from cc0b3c2 to 555e9dc Compare May 10, 2023 12:57
@tafaRU
Copy link
Member

tafaRU commented May 10, 2023 •
edited
Loading

@Borruso trattandosi di un back-port sarebbe preferibile mantenere l'autorship dei commit originali tramite un cherry-pick. Puoi provvedere?

@Borruso Borruso force-pushed the 14.0-imp-l10n_it_fatturapa_out-eur-from-journal branch 3 times, most recently from 325d57b to 20999dd Compare May 12, 2023 08:33
@Borruso Borruso force-pushed the 14.0-imp-l10n_it_fatturapa_out-eur-from-journal branch from 67067f3 to acb2b58 Compare June 30, 2023 13:16
@Borruso Borruso force-pushed the 14.0-imp-l10n_it_fatturapa_out-eur-from-journal branch from acb2b58 to 38bb9e9 Compare July 27, 2023 15:54
@SirAionTech SirAionTech mentioned this pull request Dec 13, 2023
Closed
3 tasks
@matteoopenf
Copy link
Contributor

@tafaRU e' possibiel lanciare un rebase, cosi si attiva runboat, stiamo riscontrando un problema simile e vorrei testare se questa risolve il problema

@tafaRU
Copy link
Member

tafaRU commented Dec 15, 2023

@matteoopenf prima bisogna risolvere manualmente il conflitto. @Borruso puoi provvedere? Grazie.

@matteoopenf
Copy link
Contributor

@Borruso fammi sapere se ti serve una mano che siamo interessati al fix

@Borruso Borruso force-pushed the 14.0-imp-l10n_it_fatturapa_out-eur-from-journal branch 3 times, most recently from b6205a1 to 59d25ad Compare December 18, 2023 09:01
matteoopenf
matteoopenf reviewed Dec 18, 2023
View reviewed changes
l10n_it_fatturapa_out/data/invoice_it_template.xml
<RiferimentoData t-esc="format_date(record.date)" />
</AltriDatiGestionali>
</DettaglioLinee>
</t>
</template>

<template id="account_invoice_line_it_dati_riepilogo">
Copy link
Contributor

Choose a reason for hiding this comment

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

ho notato che viene tolta questa sezione, ma non viene spostata altrove, sbaglio?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

giusto perché non viene usata

Copy link
Contributor

Choose a reason for hiding this comment

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

Grazie mille continuo a testare e tra poco approvo

@francesco-ooops francesco-ooops linked an issue Dec 18, 2023 that may be closed by this pull request
l10n_it_fatturapa_out - fattura in valuta estera con IVA #3295
Closed
3 tasks
matteoopenf
matteoopenf approved these changes Dec 19, 2023
View reviewed changes
Copy link
Contributor

@matteoopenf matteoopenf left a comment

Choose a reason for hiding this comment

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

Provata pr in locale e funziona

@matteoopenf
Copy link
Contributor

@Borruso se ti occorre aiuto per risolver il pre-commit per arrivare al merge fammelo sapere che ti faccio una pr verso la tua cosi arriviamo al merge

@Borruso Borruso force-pushed the 14.0-imp-l10n_it_fatturapa_out-eur-from-journal branch from 59d25ad to 2631b93 Compare December 19, 2023 13:23
@matteoopenf
Copy link
Contributor

@tafaRU direi che ci siamo pre-commit ok c'e' altro da fare per arrivare al merge?

@tafaRU
Copy link
Member

tafaRU commented Dec 21, 2023

@TheMule71 hai modo di fare una review tu?

@matteoopenf
Copy link
Contributor

aspettate a mergiare, dovrei aver scoperto un bug e sto cercando dov'e' applicando questa pr mi sono accorto che prezzo totale viene sbagliato

@matteoopenf
Copy link
Contributor

mi spiego qui
image
con questa patch se non ho sbagliato qualcosa, il valore indicato dalla freccia dovrebbe essere identico al prezzo unitario ma cosi non e'

@matteoopenf
Copy link
Contributor

mi spiego qui image con questa patch se non ho sbagliato qualcosa, il valore indicato dalla freccia dovrebbe essere identico al prezzo unitario ma cosi non e'

su runboat non sto riuscendo a riprodurre la cosa, sto cercando di capire se' un problema della PR o un mio problema locale, cerco di aggiornarvi in giornata

@matteoopenf
Copy link
Contributor

confermo funziona tutto ho avuto un disguido mio locale, @TheMule71 facci sapere che ne pensi anche tu e cosi possiamo procedere.
Grazie Mille

@matteoopenf
Copy link
Contributor

merge?

@francesco-ooops
Copy link
Contributor

@Borruso si può alzare un po' il codecov?

@matteoopenf
Copy link
Contributor

unica cosa che ho notato dopo che ci siamo messi questa pr in wip, che le note di credito escono con il segno errato

@matteoopenf
Copy link
Contributor

/ocabot rebase

@OCA-git-bot
Copy link
Contributor

Sorry @matteoopenf you are not allowed to rebase.

To do so you must either have push permissions on the repository, or be a declared maintainer of all modified addons.

If you wish to adopt an addon and become it's maintainer, open a pull request to add your GitHub login to the maintainers key of its manifest.

@francesco-ooops
Copy link
Contributor

/ocabot rebase

@OCA-git-bot
Copy link
Contributor

Congratulations, PR rebased to 14.0.

@OCA-git-bot OCA-git-bot force-pushed the 14.0-imp-l10n_it_fatturapa_out-eur-from-journal branch from 2631b93 to a6f7387 Compare February 29, 2024 08:55
@matteoopenf
Copy link
Contributor

/ocabot rebase

grazie

@matteoopenf
Copy link
Contributor

@Borruso sarebbe possibile fare un fix simile in modo che le note di credito escono corrette?

image

@matteoopenf
Copy link
Contributor

@Borruso ti ho fatto una pr in questo senso Borruso#19

@Borruso
Copy link
Contributor Author

Borruso commented Mar 20, 2024

#3284 ora è chiusa
#3921 c'è questa che è equivalente alla prima e quindi si dovrebbe rifare questa PR in funzione di questa nuova PR

@TheMule71 @tafaRU @matteoopenf @francesco-ooops

@francesco-ooops
Copy link
Contributor

@Borruso non ho capito :)

@matteoopenf
Copy link
Contributor

#3284 ora è chiusa #3921 c'è questa che è equivalente alla prima e quindi si dovrebbe rifare questa PR in funzione di questa nuova PR

@TheMule71 @tafaRU @matteoopenf @francesco-ooops

In sostanza vuoi fare il backport di questa #3921?

@github-actions GitHub Actions
Copy link

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Jul 21, 2024
Borruso and others added 3 commits July 26, 2024 10:50
@Borruso @TheMule71
[FIX] l10n_it_fatturapa_out: remove unused template
8767316 
Co-authored-by: TheMule71 <marco.colombo@gmail.com>
@Borruso @TheMule71
[IMP] l10n_it_fatturapa_out: use values in EUR from aml instead of co…
4f5298a 
…nverting them

Co-authored-by: TheMule71 <marco.colombo@gmail.com>
@Borruso
[IMP] l10n_it_fatturapa_out_rc: use values in EUR from aml instead of…
89dcb10 
… converting them
@Borruso Borruso force-pushed the 14.0-imp-l10n_it_fatturapa_out-eur-from-journal branch from a6f7387 to 89dcb10 Compare July 26, 2024 08:51
@github-actions github-actions bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Jul 28, 2024
@Borruso
Copy link
Contributor Author

Borruso commented Nov 8, 2024

chiusa in favore di #4177

@Borruso Borruso closed this Nov 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@matteoopenf matteoopenf matteoopenf approved these changes

Assignees
No one assigned
Labels
14.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

l10n_it_fatturapa_out - fattura in valuta estera con IVA
6 participants
@Borruso @TheMule71 @tafaRU @matteoopenf @francesco-ooops @OCA-git-bot