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

Unit Definition - Precision #1081

Merged
merged 16 commits into from
Sep 7, 2019
Merged

Conversation

solverat
Copy link
Contributor

@solverat solverat commented Sep 2, 2019

Q A
Bug fix? no
New feature? yes
BC breaks? no
Deprecations? no
Fixed tickets #1075

@dpfaffenbauer dpfaffenbauer self-requested a review September 3, 2019 10:18
@dpfaffenbauer dpfaffenbauer added this to the 2.1.0 milestone Sep 3, 2019
@solverat solverat changed the title [WIP] Unit Definition - Precision Unit Definition - Precision Sep 4, 2019
@solverat
Copy link
Contributor Author

solverat commented Sep 4, 2019

This PR allows to:

  • Add precision to each unit of measure
  • Allow adding quantity with decimals

This PR changed:

  • Use float instead of int in every quantity workflow

image

Kapture 2019-09-03 at 14 35 59

Frontend

Kapture 2019-09-03 at 16 08 04

BC Breaks

Quantity-Field is now a "text" field and needs to be watched by a javascript helper. There is also a serverside validation.

@dpfaffenbauer dpfaffenbauer merged commit 63ea7e8 into coreshop:master Sep 7, 2019
@dpfaffenbauer
Copy link
Member

thanks a lot, again.

dpfaffenbauer added a commit to dpfaffenbauer/CoreShop that referenced this pull request Sep 12, 2019
…his happens cause quantity now is a float and now a int anymore.
dpfaffenbauer added a commit that referenced this pull request Sep 13, 2019
[PriceRules] regression of #1081, force prices to be a int
@solverat solverat deleted the precisions branch October 3, 2019 14:11
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.

2 participants