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

16.0 fix issue4612 #4613

Merged
merged 1 commit into from
Feb 28, 2025
Merged

16.0 fix issue4612 #4613

merged 1 commit into from
Feb 28, 2025
+1 −0

Conversation

matteoopenf
Copy link
Contributor

No description provided.

@matteoopenf matteoopenf force-pushed the 16.0-fix_issue4612 branch 2 times, most recently from 9a37373 to 9904bf7 Compare February 10, 2025 15:13
@francesco-ooops francesco-ooops linked an issue Feb 10, 2025 that may be closed by this pull request
l10n_it_vat_registries - crashed report #4612
Closed
2 tasks
@francesco-ooops francesco-ooops mentioned this pull request Feb 10, 2025
Closed
2 tasks
@matteoopenf
Copy link
Contributor Author

i test si rompono per un problema sul modulo l10n_it_account_stamp e non per questa mia modifica

MaurizioPellegrinet
MaurizioPellegrinet approved these changes Feb 21, 2025
View reviewed changes
Copy link

@MaurizioPellegrinet MaurizioPellegrinet left a comment

Choose a reason for hiding this comment

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

Test funzionale: OK

@matteoopenf
Copy link
Contributor Author

@francesco-ooops perfavore puoi fare rebase che e' stata mergiata la pr che sistema i test?
Grazie mille

@francesco-ooops
Copy link
Contributor

/ocabot rebase

@OCA-git-bot
Copy link
Contributor

Congratulations, PR rebased to 16.0.

@francesco-ooops
Copy link
Contributor

@OCA/local-italy-maintainers qualcuno per una code review + merge? grazie!

eLBati
eLBati approved these changes Feb 27, 2025
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

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

@eLBati
Copy link
Member

eLBati commented Feb 28, 2025

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 16.0-ocabot-merge-pr-4613-by-eLBati-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit a33d8dc into OCA:16.0 Feb 28, 2025
7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 409d51e. 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
Reviewers

@eLBati eLBati eLBati approved these changes

@patrickt-oforce patrickt-oforce patrickt-oforce approved these changes

@MaurizioPellegrinet MaurizioPellegrinet MaurizioPellegrinet approved these changes

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

Successfully merging this pull request may close these issues.

l10n_it_vat_registries - crashed report
6 participants
@matteoopenf @francesco-ooops @OCA-git-bot @eLBati @patrickt-oforce @MaurizioPellegrinet