Skip to content
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

[17.0][MIG] product_configurator: Migration to 17.0 #117

Closed
wants to merge 505 commits into from

Conversation

dreispt
Copy link
Member

@dreispt dreispt commented Feb 22, 2024

Supersedes #106

@dreispt dreispt force-pushed the 17.0-mig-product_configurator branch from 4aca803 to e549ed5 Compare February 22, 2024 14:26
Pooja Khandelwal and others added 28 commits May 3, 2024 15:14
…r : add domain on value_id and value_ids and create a new field and add compute method.
…for inverse method and check weight_extra method
…ct_configurator_purchase : create test case for after or before validate the product then it id or sol is equal or not
@Vandan-OSI Vandan-OSI force-pushed the 17.0-mig-product_configurator branch 2 times, most recently from 79bf2aa to 47eaede Compare June 27, 2024 13:59
@Vandan-OSI Vandan-OSI force-pushed the 17.0-mig-product_configurator branch 2 times, most recently from 9a95e3b to 9da9898 Compare July 2, 2024 12:47
@Vandan-OSI Vandan-OSI force-pushed the 17.0-mig-product_configurator branch from 9da9898 to d268049 Compare July 18, 2024 12:50
@rousseldenis
Copy link

/ocabot migration product_configurator

@simahawk
Copy link
Contributor

@dreispt hi, what's the status here?

@dreispt
Copy link
Member Author

dreispt commented Oct 18, 2024

I don't have plans to work on this.
I just fixed commit history so that this is in a good place for people to continue with the migration.
Feel free to take over.

@simahawk
Copy link
Contributor

FYI The pandoc*.deb file has been added here. I'm updating the template here #129 and I will cleanup this PR (also removing all the improvements to be in a separated PR if they are not mandatory for 17.0)

@simahawk
Copy link
Contributor

BTW I've just found out that most of the tests are not running because they were commented on v13

image

It's the same till 16 https://github.com/OCA/product-configurator/blob/16.0/product_configurator/tests/__init__.py

😱 😱 😱 😱 😱 😱

@simahawk
Copy link
Contributor

FYI I'm cleaning up this branch, splitting things and keeping what is compatible w/ v16 to ease fwd/back porting.
I started here #133
Then I'll move on w/ this mig but I'll split out all the improvements and leave them behind for another PR to ease review and backporting. I'm doing my best to preserve ownership.

@simahawk
Copy link
Contributor

#135

@simahawk simahawk closed this Oct 24, 2024
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.