Skip to content

[19.0][MIG] stock_picking_report_custom_description: Migration to 19.0#464

Open
miguel-S73 wants to merge 28 commits intoOCA:19.0from
Studio73:19.0-mig-stock_picking_report_custom_description
Open

[19.0][MIG] stock_picking_report_custom_description: Migration to 19.0#464
miguel-S73 wants to merge 28 commits intoOCA:19.0from
Studio73:19.0-mig-stock_picking_report_custom_description

Conversation

@miguel-S73
Copy link
Contributor

Standard migration to 19.0

carlosdauden and others added 27 commits January 9, 2026 14:01
* README by fragments
* Remove models
* Fix report templates

[UPD] Update stock_picking_report_custom_description.pot

[UPD] README.rst
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: stock-logistics-reporting-16.0/stock-logistics-reporting-16.0-stock_picking_report_custom_description
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-reporting-16-0/stock-logistics-reporting-16-0-stock_picking_report_custom_description/
Currently translated at 100.0% (2 of 2 strings)

Translation: stock-logistics-reporting-16.0/stock-logistics-reporting-16.0-stock_picking_report_custom_description
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-reporting-16-0/stock-logistics-reporting-16-0-stock_picking_report_custom_description/es/
Currently translated at 100.0% (2 of 2 strings)

Translation: stock-logistics-reporting-16.0/stock-logistics-reporting-16.0-stock_picking_report_custom_description
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-reporting-16-0/stock-logistics-reporting-16-0-stock_picking_report_custom_description/it/
By default the sale description is set as
f"{product_name}\n{description". If we pass that string as the
description_picking, when we print the report, the name will be
duplicated. To avoid that effect, we want to trim the product name from
the description passed to the picking by default.

TT52044
It isn't very common to write the name in batch. Anyway it would lead to
errors in it's current shape.

TT52171
@miguel-S73 miguel-S73 mentioned this pull request Jan 12, 2026
16 tasks
Copy link

@Reyes4711-S73 Reyes4711-S73 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link

@Segui-S73 Segui-S73 left a comment

Choose a reason for hiding this comment

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

LGTM

@miguel-S73 miguel-S73 force-pushed the 19.0-mig-stock_picking_report_custom_description branch 4 times, most recently from 7535ebc to f6c3ae1 Compare January 13, 2026 09:23
@miguel-S73 miguel-S73 force-pushed the 19.0-mig-stock_picking_report_custom_description branch from f6c3ae1 to 90d412d Compare January 13, 2026 09:51
@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). 🤖

@Reyes4711-S73
Copy link

@OCA/logistics-maintainers Please, can you review/merge this PR?

@hparfr
Copy link

hparfr commented Jan 19, 2026

/ocabot migration stock_picking_report_custom_description

@OCA-git-bot OCA-git-bot added this to the 19.0 milestone Jan 19, 2026
@hparfr
Copy link

hparfr commented Jan 19, 2026

/oca merge nobump

@hparfr
Copy link

hparfr commented Jan 19, 2026

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 19.0-ocabot-merge-pr-464-by-hparfr-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Jan 19, 2026
Signed-off-by hparfr
@OCA-git-bot
Copy link
Contributor

@hparfr your merge command was aborted due to failed check(s), which you can inspect on this commit of 19.0-ocabot-merge-pr-464-by-hparfr-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.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.