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

[16.0][IMP] stock_analytic: add analytic distribution to move tree re… #583

Merged
merged 1 commit into from
Aug 11, 2023
Merged

[16.0][IMP] stock_analytic: add analytic distribution to move tree re… #583

merged 1 commit into from
Aug 11, 2023

Conversation

SergiCForgeFlow
Copy link

@SergiCForgeFlow SergiCForgeFlow commented Aug 9, 2023 •
edited
Loading

…ceipt picking view
Forward port of #564.
Field "analytic_account_id" does not longer exist in model "stock.move" so instead "analytic_distribution" field is being used.

LoisRForgeFlow
LoisRForgeFlow approved these changes Aug 11, 2023
View reviewed changes
Copy link

@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.

Tested on runbot LGTM 👍

AaronHForgeFlow
AaronHForgeFlow approved these changes Aug 11, 2023
View reviewed changes
Copy link
Contributor

@AaronHForgeFlow AaronHForgeFlow left a comment

Choose a reason for hiding this comment

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

LGTM

@AaronHForgeFlow
Copy link
Contributor

/ocabot merge minor

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 16.0-ocabot-merge-pr-583-by-AaronHForgeFlow-bump-minor, awaiting test results.

@OCA-git-bot
Copy link
Contributor

@AaronHForgeFlow The merge process could not be finalized, because command git push origin 16.0-ocabot-merge-pr-583-by-AaronHForgeFlow-bump-minor:16.0 failed with output:

To https://github.com/OCA/account-analytic
 ! [remote rejected]   16.0-ocabot-merge-pr-583-by-AaronHForgeFlow-bump-minor -> 16.0 (cannot lock ref 'refs/heads/16.0': is at f5102a607e25df807cb4ddb0fc1485358f9c8e73 but expected 978520625bb44d605ef933d04268162285e27faa)
error: failed to push some refs to 'https://***@github.com/OCA/account-analytic'

@OCA-git-bot OCA-git-bot merged commit bea8578 into OCA:16.0 Aug 11, 2023
4 checks passed
@OCA-git-bot
Copy link
Contributor

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

@LoisRForgeFlow LoisRForgeFlow deleted the 16.0-imp-stock_analytic branch August 11, 2023 07:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AaronHForgeFlow AaronHForgeFlow AaronHForgeFlow approved these changes

@LoisRForgeFlow LoisRForgeFlow LoisRForgeFlow approved these changes

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

Successfully merging this pull request may close these issues.
5 participants
@SergiCForgeFlow @AaronHForgeFlow @OCA-git-bot @LoisRForgeFlow @ThiagoMForgeFlow