Skip to content

Conversation

@joelkm
Copy link

@joelkm joelkm commented Oct 20, 2025

Backport from version 18 to version 17 of the followig modules:

  • llm
  • llm_assistant
  • llm_generate
  • llm_letta
  • llm_mcp_server
  • llm_mistral
  • llm_ollama
  • llm_openai
  • llm_replicate
  • llm_thread
  • llm_tool
  • llm_training
  • web_json_editor

Leaving it as draft. I'll ping when ready to merge

@adar2378
Copy link
Contributor

Great work!

@ayushin
Copy link
Contributor

ayushin commented Oct 20, 2025

I guess we should start writing test; but at this stage @adar2378 have you reviewed it? If so go ahead and merge and lets see if community finds any bugs #171

@ayushin ayushin mentioned this pull request Oct 20, 2025
@joelkm
Copy link
Author

joelkm commented Oct 20, 2025

@ayushin @adar2378 it is not ready to merge yet. I'm having trouble migrating llm_thread because of js backward compatibility issues.

So far you can install the app and see the chat loaded properly, but it's not integrating well with the LLM calls.

I'll ping Adar when everything is ok and ready to merge 🧐

@joelkm joelkm marked this pull request as ready for review November 7, 2025 21:32
@joelkm
Copy link
Author

joelkm commented Nov 7, 2025

@adar2378 I'm having trouble migrating the thread modules because of Owl incompatibility. I've seen you did it in 2 separate merges. Maybe we can do the same

@adar2378
Copy link
Contributor

adar2378 commented Nov 10, 2025

@joelkm Could I help you with Owl compatability somehow? Could please write a brief what's been migrated and what's missing? It would help me to know what to test and merge. Thanks again for your contribution.

@joelkm
Copy link
Author

joelkm commented Nov 13, 2025

@adar2378 llm_thread is as it was in v18. You can install the module and it'll give you no errors, but it presents a lots of issues. The first one is the mailStore service. That doesn't exist in v17 and I'm not sure how to solve it. I managed to get a workaround once with the help of Claude, but it broke the rest of the features in the process.

@adar2378
Copy link
Contributor

@joelkm I found that mail.store service is present in v17 of odoo. But indeed it's not same as 18.0
https://github.com/odoo/odoo/blob/17.0/addons/mail/static/src/core/common/store_service.js

I think if you clone both Odoo 17 and 18 source codes, then let Claude research to figure out the changes around the store then you will get better understanding I think to figure out how to solve it. Good luck and feel free to let me know if you need any other help with it.

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