Skip to content

[IMP] impersonate_login: restrict impersonate admins#954

Merged
OCA-git-bot merged 1 commit into
OCA:18.0from
PyTech-SRL:18.0-imp-impersonate_login-restrict_admin
Jun 18, 2026
Merged

[IMP] impersonate_login: restrict impersonate admins#954
OCA-git-bot merged 1 commit into
OCA:18.0from
PyTech-SRL:18.0-imp-impersonate_login-restrict_admin

Conversation

@SirPyTech

Copy link
Copy Markdown

Forward port of #866.

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hi @Kev-Roche,
some modules you are maintaining are being modified, check this out!

@SirPyTech SirPyTech marked this pull request as ready for review June 17, 2026 13:33
@SirPyTech

Copy link
Copy Markdown
Author

@gbrito and @pcastelovigo since you worked on the original PR, maybe you are interested

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

Functional ok!

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

Code review, LGTM

@Kev-Roche

Copy link
Copy Markdown
Contributor

/ocabot merge patch

@OCA-git-bot

Copy link
Copy Markdown
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 18.0-ocabot-merge-pr-954-by-Kev-Roche-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 93201ef into OCA:18.0 Jun 18, 2026
9 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

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