-
-
Notifications
You must be signed in to change notification settings - Fork 172
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
Implement m2m_to_o2m #343
base: master
Are you sure you want to change the base?
Implement m2m_to_o2m #343
Conversation
6642f7b
to
322e6b6
Compare
Co-authored-by: Rémy Taymans <[email protected]> Signed-off-by: Carmen Bianca BAKKER <[email protected]>
322e6b6
to
dba0a06
Compare
Signed-off-by: Carmen Bianca BAKKER <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure about the use case, but this will probably needs a migration from one record with m2m to several records with m2o.
Signed-off-by: Carmen Bianca BAKKER <[email protected]>
Signed-off-by: Carmen Bianca BAKKER <[email protected]>
I pushed some additional work. Specifically:
I want to see if I can create a strategy that will copy records to prevent data loss. |
Signed-off-by: Carmen Bianca BAKKER <[email protected]>
Signed-off-by: Carmen Bianca BAKKER <[email protected]>
Ready for review again. Changes since last update:
The copy strategy uses the Odoo ORM ( |
Oof, syntax error because Travis uses Python 3.6. edit: ImportError because Travis uses Python 3.6 :'( |
Signed-off-by: Carmen Bianca BAKKER <[email protected]>
Signed-off-by: Carmen Bianca BAKKER <[email protected]>
We have to switch to github actions, but not done until now (#322)... |
Signed-off-by: Carmen Bianca BAKKER <[email protected]>
Turns out Python 2 was the problem, not Python 3.6. I added a backport dependency, but it wouldn't be very difficult to get rid of the Enum and use something more primitive. edit: the backport is incomplete, ugh. I'll fix this someday soon. |
Ported over from OCA/OpenUpgrade#3911 in the hope that it can be more broadly useful.
Tagging @remytms