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_change_qty_reason #2030

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

Conversation

JasminSForgeFlow
Copy link
Contributor

Standard Migration

@ForgeFlow

rousseldenis and others added 30 commits May 10, 2024 15:05
Currently translated at 96.2% (25 of 26 strings)

Translation: stock-logistics-warehouse-12.0/stock-logistics-warehouse-12.0-stock_change_qty_reason
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-12-0/stock-logistics-warehouse-12-0-stock_change_qty_reason/fr/
Currently translated at 100.0% (26 of 26 strings)

Translation: stock-logistics-warehouse-12.0/stock-logistics-warehouse-12.0-stock_change_qty_reason
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-12-0/stock-logistics-warehouse-12-0-stock_change_qty_reason/zh_CN/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: stock-logistics-warehouse-13.0/stock-logistics-warehouse-13.0-stock_change_qty_reason
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-13-0/stock-logistics-warehouse-13-0-stock_change_qty_reason/
Currently translated at 100.0% (25 of 25 strings)

Translation: stock-logistics-warehouse-14.0/stock-logistics-warehouse-14.0-stock_change_qty_reason
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-14-0/stock-logistics-warehouse-14-0-stock_change_qty_reason/it/
@rousseldenis
Copy link
Sponsor Contributor

/ocabot migration stock_change_qty_reason

Copy link
Contributor

@LoisRForgeFlow LoisRForgeFlow left a comment

Choose a reason for hiding this comment

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

Functional review. Preset reasons work well, the free text reasons are not usable though. With preset reasons disabled, you cannot edit the reason field while doing the inv. adjustment.

Also, not sure if both reasons fields should be displayed when using preset reasons.

image

@JasminSForgeFlow
Copy link
Contributor Author

Functional review. Preset reasons work well, the free text reasons are not usable though. With preset reasons disabled, you cannot edit the reason field while doing the inv. adjustment.

Also, not sure if both reasons fields should be displayed when using preset reasons.

image

The first reason is free text when we select any reason (2nd reason) in line while doing adjustment and click on apply, it will copy reason (2nd field) value to reason (1st field), and reason (2nd field) value set as False. reason(1st field) value is for historical purpose as reason (2nd filed) will be set False when adjustment done

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.

None yet