search

OCA / social

Addons concerning odoo's social ERP features and messaging in general
https://odoo-community.org/project/99
GNU Affero General Public License v3.0
157 stars 600 forks source link

[FIX] mail_post_defer: recover "view $model" button #1382

Closed yajo closed 3 months ago

yajo commented 3 months ago

Fixes a regression in https://github.com/OCA/social/pull/1380 that made posted messages lose the "View $model" button, even if the recipient had access.

@moduon MT-6348

OCA-git-bot commented 3 months ago

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

OCA-git-bot commented 3 months ago

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

OCA-git-bot commented 3 months ago

@yajo The merge process could not start, because command git push origin 16.0-ocabot-merge-pr-1382-by-yajo-bump-patch failed with output:

To https://github.com/OCA/social
 ! [rejected]          16.0-ocabot-merge-pr-1382-by-yajo-bump-patch -> 16.0-ocabot-merge-pr-1382-by-yajo-bump-patch (fetch first)
error: failed to push some refs to 'https://***@github.com/OCA/social'
hint: Updates were rejected because the remote contains work that you do
hint: not have locally. This is usually caused by another repository pushing
hint: to the same ref. You may want to first integrate the remote changes
hint: (e.g., 'git pull ...') before pushing again.
hint: See the 'Note about fast-forwards' in 'git push --help' for details.
OCA-git-bot commented 3 months ago

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 16.0-ocabot-merge-pr-1382-by-yajo-bump-patch, awaiting test results.

OCA-git-bot commented 3 months ago

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 16.0-ocabot-merge-pr-1382-by-yajo-bump-patch, awaiting test results.

OCA-git-bot commented 3 months ago

@yajo your merge command was aborted due to failed check(s), which you can inspect on this commit of 16.0-ocabot-merge-pr-1382-by-yajo-bump-patch.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

OCA-git-bot commented 3 months ago

@yajo your merge command was aborted due to failed check(s), which you can inspect on this commit of 16.0-ocabot-merge-pr-1382-by-yajo-bump-patch.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

yajo commented 3 months ago

/ocabot merge patch

OCA-git-bot commented 3 months ago

What a great day to merge this nice PR. Let's do it! Prepared branch 16.0-ocabot-merge-pr-1382-by-yajo-bump-patch, awaiting test results.

OCA-git-bot commented 3 months ago

@yajo your merge command was aborted due to failed check(s), which you can inspect on this commit of 16.0-ocabot-merge-pr-1382-by-yajo-bump-patch.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

yajo commented 3 months ago

/ocabot merge patch

OCA-git-bot commented 3 months ago

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

OCA-git-bot commented 3 months ago

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