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

[FW][17.0][FIX] mail: Render message actions correctly with callComponent #195183

Conversation

fw-bot
Copy link
Contributor

@fw-bot fw-bot commented Jan 25, 2025

Description of the issue/feature this PR addresses:
When a custom module (e.g., OCA/social#1536) adds a new message action using callComponent, it is not invoked in certain scenarios.

Current behavior before PR:

Odoo only invokes callComponent for quick actions (2 or 3 actions). For remaining actions rendered as DropdownItem, the callComponent is not invoked.

Desired behavior after PR is merged:

This commit ensures consistent rendering behavior, invoking the callComponent for all actions, whether quick actions or dropdown items.

Complementary to: #131426

@phenix-factory @alexkuhn coud you please review this?

CC @pedrobaeza @chienandalu


I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

Forward-Port-Of: #194751
Forward-Port-Of: #194643

Currently, Odoo only renders the callComponent for quick actions (2 or 3 actions). However, the remaining actions, rendered as DropdownItem, do not invoke the callComponent.
This commit ensures that the rendering is consistent in both cases.

Complementary to: odoo#131426

X-original-commit: 86ec467
@robodoo
Copy link
Contributor

robodoo commented Jan 25, 2025

Pull request status dashboard

@robodoo robodoo added the forwardport This PR was created by @fw-bot label Jan 25, 2025
@fw-bot
Copy link
Contributor Author

fw-bot commented Jan 25, 2025

This PR targets saas-17.4 and is part of the forward-port chain. Further PRs will be created up to master.

More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port

@C3POdoo C3POdoo added the RD research & development, internal work label Jan 25, 2025
@fw-bot
Copy link
Contributor Author

fw-bot commented Jan 25, 2025

@celm1990 @alexkuhn the next pull request (#195185) is in conflict. You can merge the chain up to here by saying

@robodoo r+

More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port

@alexkuhn
Copy link
Contributor

@robodoo r+

robodoo pushed a commit that referenced this pull request Jan 25, 2025
Currently, Odoo only renders the callComponent for quick actions (2 or 3 actions). However, the remaining actions, rendered as DropdownItem, do not invoke the callComponent.
This commit ensures that the rendering is consistent in both cases.

Complementary to: #131426

closes #195183

X-original-commit: 86ec467
Signed-off-by: Alexandre Kühn (aku) <[email protected]>
@robodoo robodoo closed this Jan 25, 2025
@alexkuhn alexkuhn deleted the saas-17.4-17.0-mail-render-messageActions-with-callComponent-2jhf-fw branch January 25, 2025 22:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
forwardport This PR was created by @fw-bot RD research & development, internal work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants