Skip to content

[18.0][MIG] pos_session_sequence: Migration to 18.0 #1492

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

[18.0][MIG] pos_session_sequence: Migration to 18.0 #1492
OCA-git-bot merged 7 commits into
OCA:18.0from
APSL:18.0-mig-pos_session_sequence

Conversation

@peluko00

@peluko00 peluko00 commented Mar 4, 2026

Copy link
Copy Markdown

Migration to v18

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

@peluko00 peluko00 force-pushed the 18.0-mig-pos_session_sequence branch from ce0a2f9 to 345dd99 Compare March 4, 2026 13:53
@ivantodorovich

Copy link
Copy Markdown
Contributor

/ocabot migration pos_session_sequence

@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
return self.env.ref("point_of_sale.seq_pos_session").id

session_sequence_id = fields.Many2one(
"ir.sequence",

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
"ir.sequence",
comodel_name="ir.sequence",

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!

@peluko00 peluko00 force-pushed the 18.0-mig-pos_session_sequence branch from 345dd99 to a43f060 Compare July 2, 2026 10:22
@OCA-git-bot OCA-git-bot added series:18.0 mod:pos_session_sequence Module pos_session_sequence labels Jul 2, 2026
@peluko00 peluko00 requested a review from cubells July 2, 2026 10:22

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

@peluko00

peluko00 commented Jul 2, 2026

Copy link
Copy Markdown
Author

It's ready for merge @ivantodorovich ?

@ivantodorovich

Copy link
Copy Markdown
Contributor

Thanks!

/ocabot merge nobump

@OCA-git-bot

Copy link
Copy Markdown
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 18.0-ocabot-merge-pr-1492-by-ivantodorovich-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit a128a33 into OCA:18.0 Jul 2, 2026
6 of 7 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at 66e5430. 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.

6 participants