Skip to content

[16.0][MIG] account_move_exception: Migration to 16.0#1824

Open
enriquemartin wants to merge 1 commit intoOCA:16.0from
Digital5-Odoo:16.0-mig-account_move_exception
Open

[16.0][MIG] account_move_exception: Migration to 16.0#1824
enriquemartin wants to merge 1 commit intoOCA:16.0from
Digital5-Odoo:16.0-mig-account_move_exception

Conversation

@enriquemartin
Copy link
Copy Markdown

Due to the inactivity of the PR #1480

@enriquemartin enriquemartin force-pushed the 16.0-mig-account_move_exception branch 3 times, most recently from 207e09f to 303e615 Compare October 28, 2024 12:54
Copy link
Copy Markdown
Contributor

@luisDIXMIT luisDIXMIT left a comment

Choose a reason for hiding this comment

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

Tested locally, LGTM!

@enriquemartin enriquemartin force-pushed the 16.0-mig-account_move_exception branch from 303e615 to aa54f78 Compare April 8, 2025 12:15
@enriquemartin enriquemartin force-pushed the 16.0-mig-account_move_exception branch from aa54f78 to d3f7a9f Compare June 19, 2025 12:10
Copy link
Copy Markdown

@yibudak yibudak left a comment

Choose a reason for hiding this comment

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

Code and functional review, LGTM

@enriquemartin enriquemartin force-pushed the 16.0-mig-account_move_exception branch from d3f7a9f to 5eedce4 Compare October 16, 2025 11:48
@sergiocorato
Copy link
Copy Markdown
Contributor

/ocabot rebase

@OCA-git-bot
Copy link
Copy Markdown
Contributor

@sergiocorato The rebase process failed, because command git push --force Digital5-Odoo tmp-pr-1824:16.0-mig-account_move_exception failed with output:

remote: Permission to Digital5-Odoo/account-invoicing.git denied to OCA-git-bot.
fatal: unable to access 'https://github.com/Digital5-Odoo/account-invoicing/': The requested URL returned error: 403

Copy link
Copy Markdown
Contributor

@sergiocorato sergiocorato left a comment

Choose a reason for hiding this comment

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

I can't see the error in the logs anymore, but with this change in my environment it works, could you check?

@enriquemartin enriquemartin force-pushed the 16.0-mig-account_move_exception branch 2 times, most recently from d53cfd5 to 5c88884 Compare January 27, 2026 16:01


class TestAccountException(TransactionCase):
def setUp(self):
Copy link
Copy Markdown
Contributor

@sergiocorato sergiocorato Jan 27, 2026

Choose a reason for hiding this comment

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

This is a class

Suggested change
def setUp(self):
@classmethod
def setUpClass(cls):

etc

@enriquemartin enriquemartin force-pushed the 16.0-mig-account_move_exception branch from 6e3eec5 to cd1e297 Compare February 6, 2026 14:43
@enriquemartin enriquemartin force-pushed the 16.0-mig-account_move_exception branch from cd1e297 to 6257213 Compare March 26, 2026 09:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants