Skip to content

[18.0][FIX] pos_user_restriction: wrong partner_id in test_validate_session - #1600

Merged
OCA-git-bot merged 1 commit into
OCA:18.0from
invitu:18.0-pos_user_restriction_fix_tests
Aug 24, 2026
Merged

OCA-git-bot merged 1 commit into
OCA:18.0from
invitu:18.0-pos_user_restriction_fix_tests

Conversation

@cvinh

@cvinh cvinh commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

partner_id should be res.partner not res.users record

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hi @eLBati,
some modules you are maintaining are being modified, check this out!

@OCA-git-bot OCA-git-bot added series:18.0 mod:pos_user_restriction Module pos_user_restriction labels Aug 20, 2026
@cvinh

cvinh commented Aug 20, 2026

Copy link
Copy Markdown
Contributor Author

This PR fixes tests that failed in #1595 and #1598

@pedrobaeza pedrobaeza left a comment

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.

Clear patch, so fast-tracking it:

/ocabot merge patch

@OCA-git-bot

Copy link
Copy Markdown
Contributor

On my way to merge this fine PR!
Prepared branch 18.0-ocabot-merge-pr-1600-by-pedrobaeza-bump-patch, awaiting test results.

@OCA-git-bot
OCA-git-bot merged commit 7cafd66 into OCA:18.0 Aug 24, 2026
7 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

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

3 participants