Skip to content

[17.0][ADD] module ai_bridge_discuss_agent_base#3

Closed
arielbarreiros96 wants to merge 1 commit intoOCA:17.0from
BinhexTeam:17.0
Closed

[17.0][ADD] module ai_bridge_discuss_agent_base#3
arielbarreiros96 wants to merge 1 commit intoOCA:17.0from
BinhexTeam:17.0

Conversation

@arielbarreiros96
Copy link
Contributor

We have been preparing this module for a while now, for being able to communicate with external automation frameworks from Odoo's discuss app. The readme is still getting finishing touches, as well as the tests. But the core functionality is there. In a nutshell, this is how it works

demo.ai.discuss.mov

By using this approach agents can be used from the livechat as well, and the limit is basically defined by the quality of your external workflows.

@ValentinVinagre
Copy link

ValentinVinagre commented Jun 8, 2025

Hi @arielbarreiros96 ,
Great work. I've been reviewing it quickly, pending a more in-depth review, and I see the following:

  • I would try to make this module dependent on [ADD] ai_oca_bridge #2 (it's being developed in v16) and extend it for that connection, so we can take advantage. Another option would be to fully integrate the functionality, but I think it would be better to do it as an add-on for more decoupled migrations and maintenance.
  • It would be great not to use the "queue_job" module and avoid this dependency. Do you think it's possible to do it with triggers? That way we can remove that dependency. If it's downgraded to lower versions, it should depend on "queue_job".
  • Should it be indicated that depending on the version of Odoo, this may lead to an extension of licenses?

What do you think?
@etobella

@Enric

This comment was marked as off-topic.

@ValentinVinagre

This comment was marked as off-topic.

@Enric

This comment was marked as off-topic.

@arielbarreiros96
Copy link
Contributor Author

Hello everyone, just closed this PR since I made a silly mistake, the PR was from the base branch of the fork (17.0), this has been corrected, please let's continue here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants