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

[MIG][12.0] product_weight #449

Merged
merged 8 commits into from
Aug 20, 2019
Merged

Conversation

tbaden
Copy link
Member

@tbaden tbaden commented Apr 26, 2019

Migration of product_weight to v12

@tbaden tbaden mentioned this pull request Apr 26, 2019
24 tasks
@pedrobaeza pedrobaeza added this to the 12.0 milestone Apr 26, 2019
@tbaden tbaden changed the title [MIG][12.0] product_weight [MIG][12.0][WIP] product_weight Apr 26, 2019
@tbaden tbaden changed the title [MIG][12.0][WIP] product_weight [MIG][12.0] product_weight Apr 26, 2019
@oca-clabot
Copy link

Hey @tbaden, thank you for your Pull Request.

It looks like some users haven't signed our Contributor License Agreement, yet.
You can read and sign our full Contributor License Agreement here: http://odoo-community.org/page/cla
Here is a list of the users:

Appreciation of efforts,
OCA CLAbot

@tbaden
Copy link
Member Author

tbaden commented May 17, 2019

added missing commit history

@tbaden
Copy link
Member Author

tbaden commented May 17, 2019

@pedrobaeza runbot is getting killed after 40 minutes without valid logs to check.
do you have any idea?

@pedrobaeza
Copy link
Member

There's a problem right now with the sizing of the servers. Let's see if @gurneyalex can fix it soon.

Meanwhile, force a rebuild to see if you can get it.

Copy link

@florian-dacosta florian-dacosta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi,
Thanks for the PR.
There is a bug in case of multi level bom if the bom lines have a quantity different than 1
More explanation and the fix here : tbaden#1

@tbaden
Copy link
Member Author

tbaden commented Jul 2, 2019

@florian-dacosta thanks for the information, I'll attend these changes at the end of this week.

@pedrobaeza
Copy link
Member

Any news on this?

@tbaden
Copy link
Member Author

tbaden commented Aug 19, 2019

@pedrobaeza is that the right way to merge the fix into the PR?
btw. thanks for the reminder, somehow missed this one

@pedrobaeza
Copy link
Member

Yes, but I see one line that can be removed commented on the original PR.

@tbaden
Copy link
Member Author

tbaden commented Aug 19, 2019

fixed it.

Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Amended commit history and changes seems OK, so counting with implicit @florian-dacosta approval, I'm merging.

@pedrobaeza
Copy link
Member

/ocabot merge

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Rebased to 12.0-ocabot-merge-pr-449-by-pedrobaeza-bump-no, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Aug 20, 2019
Signed-off-by pedrobaeza
@OCA-git-bot OCA-git-bot merged commit d1e785b into OCA:12.0 Aug 20, 2019
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 6c54a01. Thanks a lot for contributing to OCA. ❤️

PS: Don't worry if GitHub says there are unmerged commits: it is due to a rebase before merge. All commits of this PR have been merged into 12.0.

QuocDuong1306 pushed a commit to QuocDuong1306/product-attribute that referenced this pull request Aug 9, 2023
Signed-off-by pedrobaeza
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.

7 participants