Skip to content

[17.0][FIX]stock_move_value_report: fix stock_move_line_value_report not printing stock move lines#476

Merged
OCA-git-bot merged 1 commit into
OCA:17.0from
ForgeFlow:17.0-fix-stock_move_value_report-fix_report_stock_moves
Jun 5, 2026
Merged

[17.0][FIX]stock_move_value_report: fix stock_move_line_value_report not printing stock move lines#476
OCA-git-bot merged 1 commit into
OCA:17.0from
ForgeFlow:17.0-fix-stock_move_value_report-fix_report_stock_moves

Conversation

@GuillermoNforgeflow

@GuillermoNforgeflow GuillermoNforgeflow commented Feb 20, 2026 •
edited
Loading

Copy link
Copy Markdown

Now the report prints all selected stock move lines, removing the picking and scrap filters and computing values directly from the related stock moves.

[FIX]stock_move_value_report: fix stock_move_line_value_report not pr…
a02d4d5 
…inting stock move lines

Now the report prints all selected stock move lines, removing the picking and scrap filters and computing values directly from the related stock moves.
@GuillermoNforgeflow GuillermoNforgeflow force-pushed the 17.0-fix-stock_move_value_report-fix_report_stock_moves branch from 64a30a9 to a02d4d5 Compare February 23, 2026 07:34
@GuillermoNforgeflow GuillermoNforgeflow changed the title [17.0][FIX]stock_move_value_report: fix stock_move_line_value_report not pinting stock move lines [17.0][FIX]stock_move_value_report: fix stock_move_line_value_report not printing stock move lines Feb 24, 2026
GuillemCForgeFlow
GuillemCForgeFlow approved these changes Feb 24, 2026
View reviewed changes

@GuillemCForgeFlow GuillemCForgeFlow left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

makes sense, this way we can also include other moves, such as the repair ones 👍🏿

@GuillemCForgeFlow

GuillemCForgeFlow commented Mar 11, 2026

Copy link
Copy Markdown

@chienandalu @CarlosRoca13 @BhaveshHeliconia could you kindly review this fix we introduced to include repair related stock moves?

@GuillemCForgeFlow

GuillemCForgeFlow commented Mar 17, 2026

Copy link
Copy Markdown

@gurneyalex any chance you can get this fix merged? 🙂 thanks in advance for your time

rrebollo
rrebollo approved these changes Mar 23, 2026
View reviewed changes

@rrebollo rrebollo left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review. LGTM! It Looks simpler now 👍.
Would you be so kind to review my #454 back?

@OCA-git-bot

OCA-git-bot commented Mar 23, 2026

Copy link
Copy Markdown
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). 🤖

AndreuOForgeFlow
AndreuOForgeFlow approved these changes Jun 5, 2026
View reviewed changes

@AndreuOForgeFlow AndreuOForgeFlow left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM!

@AndreuOForgeFlow

AndreuOForgeFlow commented Jun 5, 2026

Copy link
Copy Markdown

Hi @pedrobaeza, could we perhaps merge this one?

@pedrobaeza pedrobaeza added this to the 17.0 milestone Jun 5, 2026
@pedrobaeza

pedrobaeza commented Jun 5, 2026

Copy link
Copy Markdown
Member

/ocabot merge patch

@OCA-git-bot

OCA-git-bot commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 17.0-ocabot-merge-pr-476-by-pedrobaeza-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 3e59031 into OCA:17.0 Jun 5, 2026
7 checks passed
@OCA-git-bot

OCA-git-bot commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

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

@AndreuOForgeFlow AndreuOForgeFlow deleted the 17.0-fix-stock_move_value_report-fix_report_stock_moves branch June 5, 2026 10:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

4 more reviewers

@rrebollo rrebollo rrebollo approved these changes

@GuillemCForgeFlow GuillemCForgeFlow GuillemCForgeFlow approved these changes

@AndreuOForgeFlow AndreuOForgeFlow AndreuOForgeFlow approved these changes

@BhaveshHeliconia BhaveshHeliconia BhaveshHeliconia approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

approved merged 🎉 ready to merge

Projects

None yet

Milestone

17.0

Development

Successfully merging this pull request may close these issues.

7 participants

@GuillermoNforgeflow @GuillemCForgeFlow @OCA-git-bot @AndreuOForgeFlow @pedrobaeza @rrebollo @BhaveshHeliconia