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][ADD] new module purchase_stock_analytic #333

Merged
merged 3 commits into from
Aug 10, 2022
Merged

[13.0][ADD] new module purchase_stock_analytic #333

merged 3 commits into from
Aug 10, 2022

Conversation

baimont
Copy link
Contributor

@baimont baimont commented Nov 10, 2020

New module that copies the analytic account and the analytic tags of the purchase order item to the stock move

Depends on #332

@baimont baimont force-pushed the bai_13.0_purchase_stock_analytic branch from 7364e03 to 23515c5 Compare November 10, 2020 10:03
@sanrav
Copy link
Contributor

sanrav commented Nov 11, 2020

Functional testing OK

sanrav
sanrav approved these changes Nov 11, 2020
View reviewed changes
Copy link
Contributor

@sanrav sanrav left a comment

Choose a reason for hiding this comment

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

Functional testing OK

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@baimont baimont force-pushed the bai_13.0_purchase_stock_analytic branch from 23515c5 to c7bb1b4 Compare November 26, 2020 12:10
MeetKD
MeetKD suggested changes Nov 27, 2020
View reviewed changes
Copy link

@MeetKD MeetKD left a comment

Choose a reason for hiding this comment

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

Minor Changes !

purchase_stock_analytic/__manifest__.py Outdated Show resolved Hide resolved
purchase_stock_analytic/models/purchase_order_line.py Outdated Show resolved Hide resolved
@baimont baimont force-pushed the bai_13.0_purchase_stock_analytic branch from c7bb1b4 to 1501657 Compare January 27, 2021 13:47
@github-actions
Copy link

github-actions bot commented Apr 3, 2022

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Apr 3, 2022
@rousseldenis
Copy link
Contributor

@MeetKD

@github-actions github-actions bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Apr 17, 2022
@rousseldenis
Copy link
Contributor

@OCA/accounting-maintainers

@dreispt
Copy link
Member

dreispt commented Aug 10, 2022

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 13.0-ocabot-merge-pr-333-by-dreispt-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Aug 10, 2022
@OCA-git-bot
Merge PR #333 into 13.0
32ee9a2 
Signed-off-by dreispt
@OCA-git-bot
Copy link
Contributor

It looks like something changed on 13.0 in the meantime.
Let me try again (no action is required from you).
Prepared branch 13.0-ocabot-merge-pr-333-by-dreispt-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit aa971ea into OCA:13.0 Aug 10, 2022
@OCA-git-bot
Copy link
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sanrav sanrav sanrav approved these changes

@MeetKD MeetKD MeetKD approved these changes

@acsonefho acsonefho acsonefho approved these changes

@rousseldenis rousseldenis rousseldenis approved these changes

@apineux apineux apineux 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.
8 participants
@baimont @sanrav @OCA-git-bot @rousseldenis @dreispt @apineux @acsonefho @MeetKD