Skip to content

[18.0][MIG] pos_cash_control_extension: Migration to 18.0 #1491

Merged
OCA-git-bot merged 6 commits into
OCA:18.0from
APSL:18.0-mig-pos_cash_control_extension
Jul 2, 2026
Merged

[18.0][MIG] pos_cash_control_extension: Migration to 18.0 #1491
OCA-git-bot merged 6 commits into
OCA:18.0from
APSL:18.0-mig-pos_cash_control_extension

Conversation

@peluko00

@peluko00 peluko00 commented Mar 4, 2026

Copy link
Copy Markdown

Migration to v18

cc https://github.com/APSL 36689
@miquelalzanillas @javierobcn @mpascuall @BernatObrador @ppyczko @lbarry-apsl please review

@ivantodorovich ivantodorovich left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

/ocabot migration pos_cash_control_extension

# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).

from odoo import models
from odoo import _, fields, models

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

use self.env._ instead of _

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Fixed!

@OCA-git-bot OCA-git-bot added this to the 18.0 milestone Apr 7, 2026
@OCA-git-bot OCA-git-bot mentioned this pull request Apr 7, 2026
34 tasks
@peluko00 peluko00 force-pushed the 18.0-mig-pos_cash_control_extension branch from c5b93a2 to c638911 Compare April 8, 2026 06:17

@palomagrc93 palomagrc93 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@peluko00

peluko00 commented Jul 2, 2026

Copy link
Copy Markdown
Author

It's ready for merge @ivantodorovich ?

@peluko00 peluko00 force-pushed the 18.0-mig-pos_cash_control_extension branch from c638911 to 1c0a3de Compare July 2, 2026 10:56
@OCA-git-bot OCA-git-bot added series:18.0 mod:pos_cash_control_extension Module pos_cash_control_extension labels Jul 2, 2026

@shirashi3771 shirashi3771 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM! Code review and tested in runboat .

@ivantodorovich

Copy link
Copy Markdown
Contributor

Thanks!

/ocabot merge nobump

@OCA-git-bot

Copy link
Copy Markdown
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 18.0-ocabot-merge-pr-1491-by-ivantodorovich-bump-nobump, awaiting test results.

@OCA-git-bot

Copy link
Copy Markdown
Contributor

It looks like something changed on 18.0 in the meantime.
Let me try again (no action is required from you).
Prepared branch 18.0-ocabot-merge-pr-1491-by-ivantodorovich-bump-nobump, awaiting test results.

@OCA-git-bot

Copy link
Copy Markdown
Contributor

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

@OCA-git-bot OCA-git-bot merged commit 16e7a43 into OCA:18.0 Jul 2, 2026
5 of 7 checks passed
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.

6 participants