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.

Sign up for GitHub

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

Jump to bottom

[13.0] stock_cycle_count error on location_id #1185

Merged
merged 1 commit into from Sep 30, 2021
Merged

[13.0] stock_cycle_count error on location_id #1185

merged 1 commit into from Sep 30, 2021

Conversation

celm1990
Copy link
Sponsor

Hi, when i try open Accuracy Stats get following error
image

image

field in stock.inventory is location_ids(many2many) https://github.com/odoo/odoo/blob/13.0/addons/stock/models/stock_inventory.py#L45-L49

@celm1990
Copy link
Sponsor Author

celm1990 commented Sep 13, 2021 •
edited

someone help me with this PR please?

CC: @LoisRForgeFlow @jimhoefnagels

@pedrobaeza pedrobaeza added this to the 13.0 milestone Sep 14, 2021
@celm1990 celm1990 force-pushed the 13.0-fix-stock_cycle_count branch 2 times, most recently from 3219e72 to 1ab7047 Compare September 21, 2021 05:30
@celm1990
Copy link
Sponsor Author

@pedrobaeza changes are OK? or I need adapt something

LoisRForgeFlow
LoisRForgeFlow approved these changes Sep 30, 2021
View reviewed changes
Copy link
Contributor

@LoisRForgeFlow LoisRForgeFlow left a comment

Choose a reason for hiding this comment

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

Thanks

@LoisRForgeFlow
Copy link
Contributor

LoisRForgeFlow commented Sep 30, 2021 •
edited

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 13.0-ocabot-merge-pr-1185-by-LoisRForgeFlow-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Sep 30, 2021
@OCA-git-bot
Merge PR #1185 into 13.0
0862baa 
Signed-off-by LoisRForgeFlow
@OCA-git-bot
Copy link
Contributor

@LoisRForgeFlow your merge command was aborted due to failed check(s), which you can inspect on this commit of 13.0-ocabot-merge-pr-1185-by-LoisRForgeFlow-bump-nobump.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

@LoisRForgeFlow
Copy link
Contributor

wrong command, not sure if editing it was enough so I restart the merge:

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 13.0-ocabot-merge-pr-1185-by-LoisRForgeFlow-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 8479fa6 into OCA:13.0 Sep 30, 2021
@OCA-git-bot
Copy link
Contributor

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

@celm1990 celm1990 deleted the 13.0-fix-stock_cycle_count branch September 30, 2021 18:02
@LoisRForgeFlow LoisRForgeFlow mentioned this pull request Oct 1, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pedrobaeza pedrobaeza pedrobaeza approved these changes

@LoisRForgeFlow LoisRForgeFlow LoisRForgeFlow approved these changes

Assignees
No one assigned
Labels
approved merged 🎉
Projects
None yet
Milestone
13.0
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@celm1990 @LoisRForgeFlow @OCA-git-bot @pedrobaeza