search

OCA / l10n-italy

Odoo Italian localization
https://www.odoo-italia.org
GNU Affero General Public License v3.0
142 stars 297 forks source link

[FIX] Import zip file e-fatture non possibile se non si hanno i diritti da Amministratore. Il sistema rimporta errore generico #4107

Closed SirAionTech closed 1 month ago

SirAionTech commented 2 months ago

Risolve https://github.com/OCA/l10n-italy/issues/3336, https://github.com/OCA/l10n-italy/issues/3623 e https://github.com/OCA/l10n-italy/pull/4100 per 16.0.

SirAionTech commented 1 month ago

Fatto rebase per risolvere i conflitti

SirAionTech commented 1 month ago

Fatto rebase per risolvere i conflitti

OCA-git-bot commented 1 month ago

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

francesco-ooops commented 1 month ago

@TheMule71 @eLBati qualcosa in contrario al merge?

SirAionTech commented 1 month ago

@OCA/local-italy-maintainers è https://github.com/OCA/l10n-italy/labels/ready%20to%20merge, si può mergiare?

francesco-ooops commented 1 month ago

@TheMule71 che ne pensi di questa? E' un problema sui diritti di accesso che ci portiamo dietro da un sacco di tempo, mi piacerebbe risolverlo :)

OpenCode commented 1 month ago

@OCA/local-italy-maintainers è ready to merge , si può mergiare?

Preferisci patch o minor per il versionamento?

SirAionTech commented 1 month ago

@OCA/local-italy-maintainers è ready to merge , si può mergiare?

Preferisci patch o minor per il versionamento?

Essendoci degli script di migrazione penso sia meglio nobump

OpenCode commented 1 month ago

@OCA/local-italy-maintainers è ready to merge , si può mergiare?

Preferisci patch o minor per il versionamento?

Essendoci degli script di migrazione penso sia meglio nobump

l10n_it_fatturapa non ha bump però in quel caso. Vuoi aggiungere anche quello?

SirAionTech commented 1 month ago

@OCA/local-italy-maintainers è ready to merge , si può mergiare?

Preferisci patch o minor per il versionamento?

Essendoci degli script di migrazione penso sia meglio nobump

l10n_it_fatturapa non ha bump però in quel caso. Vuoi aggiungere anche quello?

Sì giusto grazie, l'ho aggiunto insieme al rename di una variabile

francesco-ooops commented 1 month ago

@SirAionTech ci evidenzi la modifica del rename su #4194 così la portiamo anche di là?

OpenCode commented 1 month ago

/ocabot merge nobump

OCA-git-bot commented 1 month ago

This PR looks fantastic, let's merge it! Prepared branch 16.0-ocabot-merge-pr-4107-by-OpenCode-bump-nobump, awaiting test results.

OCA-git-bot commented 1 month ago

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