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

[MIG] l10n_it_corrispettivi_fatturapa_out #2979

Merged
merged 8 commits into from
May 13, 2023
Merged

[MIG] l10n_it_corrispettivi_fatturapa_out #2979

merged 8 commits into from
May 13, 2023

Conversation

SirTakobi
Copy link
Contributor

@SirTakobi SirTakobi commented Oct 24, 2022 •
edited
Loading

Da https://github.com/OCA/l10n-italy/tree/0b7b3c7be0b77ae20ec35997687c28ce5d437773/l10n_it_corrispettivi_fatturapa_out.
Il modulo in oggetto in pratica nascondeva il pulsante di esportazione della FE nel form dei corrispettivi.

Ora che le ricevute sono nel core di Odoo (vedi in #2722 il collegamento con i corrispettivi), penso sia responsabilità di l10n_it_fatturapa_out mostrare i pulsanti per le FE solo nelle fatture in uscita.

@SirTakobi SirTakobi mentioned this pull request Oct 24, 2022
Open
76 tasks
@SirTakobi SirTakobi marked this pull request as ready for review October 24, 2022 09:46
@primes2h
Copy link
Contributor

primes2h commented Nov 8, 2022

/ocabot migration l10n_it_corrispettivi_fatturapa_out

@OCA-git-bot OCA-git-bot added this to the 14.0 milestone Nov 8, 2022
@eLBati eLBati mentioned this pull request Feb 1, 2023
Closed
1 task
@SirTakobi SirTakobi force-pushed the 14.0-mig-l10n_it_corrispettivi_fatturapa_out branch from c775964 to 34aad2e Compare March 20, 2023 12:32
@SirTakobi SirTakobi force-pushed the 14.0-mig-l10n_it_corrispettivi_fatturapa_out branch from 34aad2e to d6724d4 Compare April 13, 2023 14:02
@SirTakobi SirTakobi force-pushed the 14.0-mig-l10n_it_corrispettivi_fatturapa_out branch from d6724d4 to 8b0d6a4 Compare April 26, 2023 12:51
mrcast
mrcast approved these changes Apr 26, 2023
View reviewed changes
Copy link
Contributor

@mrcast mrcast left a comment

Choose a reason for hiding this comment

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

Functional test: OK!

eLBati
eLBati approved these changes May 13, 2023
View reviewed changes
Copy link
Member

@eLBati eLBati left a comment

Choose a reason for hiding this comment

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

Grazie

@eLBati
Copy link
Member

eLBati commented May 13, 2023

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 14.0-ocabot-merge-pr-2979-by-eLBati-bump-nobump, awaiting test results.

@OCA-git-bot
Copy link
Contributor

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). 🤖

@OCA-git-bot OCA-git-bot merged commit ad9362f into OCA:14.0 May 13, 2023
@OCA-git-bot
Copy link
Contributor

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

@matteoopenf
Copy link
Contributor

matteoopenf commented Jul 5, 2023 •
edited
Loading

ho appena notato che mergiando questa pr si e' introdotto un problema e ora apro una issue collegata.
image

Non si esportano e non viene piu' mostrato il bottone esporta fattura elettronica per le note di credito, e relative tab annesse.
Erroneamente e' stato messo in_refound anziche out_refund

@matteoopenf matteoopenf mentioned this pull request Jul 5, 2023
Closed
2 tasks
@SirAionTech SirAionTech mentioned this pull request Jan 4, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eLBati eLBati eLBati approved these changes

@mrcast mrcast mrcast approved these changes

Assignees
No one assigned
Labels
approved merged 🎉 migration ready to merge
Projects
None yet
Milestone
14.0
Development

Successfully merging this pull request may close these issues.
8 participants
@SirTakobi @primes2h @eLBati @OCA-git-bot @matteoopenf @mrcast @francesco-ooops @monen17