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.

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

[17.0][MIG]stock_picking_info_lot: Migration to 17.0 #1689

Open
wants to merge 15 commits into
base: 17.0
Choose a base branch
from

Conversation

SodexisTeam
Copy link
Member

No description provided.

ZavarinArtem and others added 14 commits August 30, 2024 18:04
[ADD] added readme folder and content

[IMP] check with pre-commits

[FIX] renamed views file to follow guideline

[15.0][ADD] added setup files

[15.0][ADD] added setup files

[15.0][IMP] re-write DESCRIPTION.rst

[IMP] stock_picking_info_lot: require lot info upon validation

[ADD] added unit test for validation check

[ADD] recreating setup files

[ADD] recreating setup files

[ADD] added ROADMAP.rst
Currently translated at 100.0% (9 of 9 strings)

Translation: stock-logistics-workflow-16.0/stock-logistics-workflow-16.0-stock_picking_info_lot
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-workflow-16-0/stock-logistics-workflow-16-0-stock_picking_info_lot/es/
Currently translated at 77.7% (7 of 9 strings)

Translation: stock-logistics-workflow-16.0/stock-logistics-workflow-16.0-stock_picking_info_lot
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-workflow-16-0/stock-logistics-workflow-16-0-stock_picking_info_lot/it/
Currently translated at 100.0% (9 of 9 strings)

Translation: stock-logistics-workflow-16.0/stock-logistics-workflow-16.0-stock_picking_info_lot
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-workflow-16-0/stock-logistics-workflow-16-0-stock_picking_info_lot/it/
@SodexisTeam SodexisTeam mentioned this pull request Sep 2, 2024
53 tasks
@rousseldenis
Copy link
Sponsor Contributor

/ocabot migration stock_picking_info_lot

Copy link
Sponsor Contributor

@rousseldenis rousseldenis left a comment

Choose a reason for hiding this comment

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

One change needed

)
if lines_missing_lotinfo:
raise exceptions.UserError(
_("Missing Lot Info for Products %s.")
Copy link
Sponsor Contributor

Choose a reason for hiding this comment

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

Please use translation function arguments instead.

@SodexisTeam SodexisTeam force-pushed the 17.0-mig-stock_picking_info_lot branch 3 times, most recently from d1f1a5e to b4a03b5 Compare September 24, 2024 06:25
Copy link
Sponsor Contributor

@rousseldenis rousseldenis left a comment

Choose a reason for hiding this comment

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

LGTM

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.

8 participants