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

[Fix] change_volume_dimension_unit_to_inch #92478

Open
wants to merge 1 commit into
base: 15.0
Choose a base branch
from

Conversation

angv-odoo
Copy link
Contributor

@angv-odoo angv-odoo commented May 31, 2022

Due to the fact that the only avaliable volume unit in odoo 15 is feet for imperial system,
Users can not create packages that have less then 1 foot dimension. This commit changes feet
to inches, so the lowest volume for package can be 1 inch^3 instead of 1 foot^3

--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

@robodoo
Copy link
Contributor

robodoo commented May 31, 2022

@angv-odoo angv-odoo force-pushed the 15.0-add_new_volume_dimension_units-angv branch from 166ff58 to ce37f6e Compare May 31, 2022 08:48
Due to the fact that the only avaliable volume unit in odoo 15 is feet for imperial system,
Users can not create packages that have less then 1 foot dimension. This commit changes feet
to inches, so the lowest value for package can be 1 inch^3 instead of 1 foot^3
@angv-odoo angv-odoo force-pushed the 15.0-add_new_volume_dimension_units-angv branch from ce37f6e to 3bc4d20 Compare May 31, 2022 08:49
@C3POdoo C3POdoo added the RD research & development, internal work label May 31, 2022
@angv-odoo angv-odoo marked this pull request as ready for review May 31, 2022 11:28
@angv-odoo angv-odoo changed the base branch from 15.0 to master June 1, 2022 09:09
@angv-odoo angv-odoo changed the base branch from master to 15.0 June 1, 2022 09:10
@amoyaux
Copy link
Contributor

amoyaux commented Jun 14, 2022

Ok but I would avoid in stable. I agree with the change if we stuck to delivery but we don't know if some other people used this field for other purpose (customization). If there is a lot of ticket/issues linked to it, then we could do it

Also I notice that the package size is not converted from cm to inch in easypost, usps (less important since it's limited from USA to USA)
Carrier where dimension unit can be choose: There is no conversion if the package and carrier dimension units are not the same (we should either do it or make them read only)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RD research & development, internal work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants