Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[16.0][FIX] mass_mailing_custom_unsubscribe: portal user permissions #1571

Conversation

chienandalu
Copy link
Member

Portal users need the same permissions as public ones to read the unsubscribe reasons.

cc @Tecnativa TT54814

please review @pilarvargas-tecnativa @pedrobaeza

@pedrobaeza pedrobaeza added this to the 16.0 milestone Feb 3, 2025
@pedrobaeza
Copy link
Member

Again a signature change in mail_composer_cc_bcc? Please deactivate its tests, as they are always problematic.

@pedrobaeza
Copy link
Member

Still red...

@chienandalu
Copy link
Member Author

Bad timing 😅 odoo/odoo@a78d75b

Portal users need the same permissions as public ones to read the
unsubscribe reasons.

TT54814
@chienandalu chienandalu force-pushed the 16.0-fix-mass_mailing_custom_unsubscribe-unsubscribe-portal branch from cf9b7be to 17e2d76 Compare February 4, 2025 15:29
@chienandalu
Copy link
Member Author

Green at last

@pedrobaeza
Copy link
Member

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

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

@OCA-git-bot OCA-git-bot merged commit af1d0e9 into OCA:16.0 Feb 4, 2025
8 of 9 checks passed
@OCA-git-bot
Copy link
Contributor

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

@pedrobaeza pedrobaeza deleted the 16.0-fix-mass_mailing_custom_unsubscribe-unsubscribe-portal branch February 4, 2025 16:03
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.

5 participants