Skip to content

[18.0][ADD] helpdesk_mgmt_livechat: add new module - #954

Open
CristianoMafraJunior wants to merge 1 commit into
OCA:18.0from
Escodoo:18.0-add-helpdesk_mgmt_livechat
Open

CristianoMafraJunior wants to merge 1 commit into
OCA:18.0from
Escodoo:18.0-add-helpdesk_mgmt_livechat

Conversation

@CristianoMafraJunior

@CristianoMafraJunior CristianoMafraJunior commented Mar 3, 2026 •

Copy link
Copy Markdown
Member

@CristianoMafraJunior
CristianoMafraJunior force-pushed the 18.0-add-helpdesk_mgmt_livechat branch 2 times, most recently from cd6c34e to dcfbe96 Compare March 4, 2026 11:30

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

LGTM

@OCA-git-bot

Copy link
Copy Markdown
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@CristianoMafraJunior
CristianoMafraJunior force-pushed the 18.0-add-helpdesk_mgmt_livechat branch from 6c22dd7 to 07c650d Compare July 21, 2026 12:06
@marcelsavegnago

Copy link
Copy Markdown
Member

ping @dalonsod @etobella

@cgarcia-solvos cgarcia-solvos 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 and Functional review:

/ticket command not working:

Comment on lines +3 to +23
import {insert} from "@mail/model/model_field_command";
import {registerPatch} from "@mail/model/model_core";

registerPatch({
name: "MessagingInitializer",
recordMethods: {
/**
* @override
*/
_initCommands() {
this._super();
this.messaging.update({
commands: insert({
help: this.env._t("Create a new ticket (/ticket ticket title)"),
methodName: "execute_command_ticket",
name: "ticket",
}),
});
},
},
});

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In older Odoo versions, commands were registered by patching MessagingInitializer via registerPatch (@mail/model/model_core, @mail/model/model_field_command). In modern versions, commands are registered directly via registry.category("discuss.channel_commands").

Suggested change
import {insert} from "@mail/model/model_field_command";
import {registerPatch} from "@mail/model/model_core";
registerPatch({
name: "MessagingInitializer",
recordMethods: {
/**
* @override
*/
_initCommands() {
this._super();
this.messaging.update({
commands: insert({
help: this.env._t("Create a new ticket (/ticket ticket title)"),
methodName: "execute_command_ticket",
name: "ticket",
}),
});
},
},
});
import {_t} from "@web/core/l10n/translation";
import {registry} from "@web/core/registry";
registry.category("discuss.channel_commands").add("ticket", {
help: _t("Create a new ticket (/ticket ticket title)"),
methodName: "execute_command_ticket",
});

Comment on lines +20 to +24
"assets": {
"mail.assets_messaging": [
"helpdesk_mgmt_livechat/static/src/models/*.esm.js",
],
},

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"assets": {
"mail.assets_messaging": [
"helpdesk_mgmt_livechat/static/src/models/*.esm.js",
],
},
"assets": {
"web.assets_backend": [
"helpdesk_mgmt_livechat/static/src/models/*.esm.js",
],
},

@marcelsavegnago

Copy link
Copy Markdown
Member

Code and Functional review:

/ticket command not working:

ping @CristianoMafraJunior

@OCA-git-bot

Copy link
Copy Markdown
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

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