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.

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

[12.0] [FIX] l10n_it_account_stamp_sale: change module dependency #3105

Conversation

primes2h
Copy link
Contributor

@primes2h primes2h commented Dec 28, 2022

Il modulo l10n_it_account_stamp_sale permette di gestire l'applicabilità dell'imposta di bollo nelle fatture provenienti da un ordine di vendita.
Dato che gli ordini di vendita vengono aggiunti installando sale_management (applicazione "Vendite"), il modulo dovrebbe di conseguenza dipendendere da quest'ultimo e non da sale.

La situazione attuale porta al seguente problema.
In una installazione Odoo con ad es. fatturazione elettronica, imposta di bollo e dichiarazione di intento (per obbligo dei clienti), il modulo l10n_it_account_stamp_sale viene installato in modo automatico anche se non necessario.

Questa PR corregge il problema.

Vedi anche #3106

@OCA-git-bot
Copy link
Contributor

Hi @tafaRU,
some modules you are maintaining are being modified, check this out!

@eLBati
Copy link
Member

eLBati commented Jan 9, 2023

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 12.0-ocabot-merge-pr-3105-by-eLBati-bump-patch, 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 9b4d96f into OCA:12.0 Jan 9, 2023
@OCA-git-bot
Copy link
Contributor

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

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