Skip to content

[6180][MIG] stock_outgoing_shipment_report: Migration to 18.0#238

Merged
kanda999 merged 15 commits into18.0from
18.0-mig-stock_outgoing_shipment_report
Mar 14, 2026
Merged

[6180][MIG] stock_outgoing_shipment_report: Migration to 18.0#238
kanda999 merged 15 commits into18.0from
18.0-mig-stock_outgoing_shipment_report

Conversation

@AungKoKoLin1997
Copy link
Contributor

@AungKoKoLin1997
Copy link
Contributor Author

Need to work on note field since this field was removed from stock.move.

@yostashiro
Copy link
Member

Need to work on note field since this field was removed from stock.move.

@AungKoKoLin1997 Let's use https://github.com/OCA/sale-workflow/tree/18.0/sale_order_line_note to add a note field to sale.order.line, and let this module depend on it to take the value from the field. I don't think we need to have the note field in stock.move if it's only used for the memo field in the report output.

@AungKoKoLin1997 AungKoKoLin1997 force-pushed the 18.0-mig-stock_outgoing_shipment_report branch from 66cb39c to d8e2588 Compare December 3, 2025 01:41
@yostashiro yostashiro changed the title [6180][MIG] stock_outgoing_shipping_report: Migration to 18.0 [6180][MIG] stock_outgoing_shipment_report: Migration to 18.0 Dec 12, 2025
@yostashiro yostashiro force-pushed the 18.0-mig-stock_outgoing_shipment_report branch from 93efd58 to c4163bd Compare December 12, 2025 15:06
Copy link
Member

@yostashiro yostashiro left a comment

Choose a reason for hiding this comment

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

@AungKoKoLin1997 Please review my last commit and squash it into the migration commit.

@AungKoKoLin1997 AungKoKoLin1997 force-pushed the 18.0-mig-stock_outgoing_shipment_report branch from c4163bd to f988b62 Compare December 15, 2025 03:48
@AungKoKoLin1997
Copy link
Contributor Author

@AungKoKoLin1997 Please review my last commit and squash it into the migration commit.

@yostashiro I reviewed the code and functional.
It looks good and I squash the commits.

@kanda999 kanda999 merged commit 77bae8e into 18.0 Mar 14, 2026
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants