-
Notifications
You must be signed in to change notification settings - Fork 92
17.0 Backport migration #171
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
base: 17.0-backport-migration
Are you sure you want to change the base?
Conversation
|
Great work! |
|
@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 🧐 |
|
@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 |
|
@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. |
|
@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. |
|
@joelkm I found that mail.store service is present in v17 of odoo. But indeed it's not same as 18.0 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. |
Backport from version 18 to version 17 of the followig modules:
Leaving it as draft. I'll ping when ready to merge