Merge pull request #754 from mirumee/fix/order-draft-negative-quantity
Fix order draft negative quantity
This commit is contained in:
commit
b0a416e0fa
2 changed files with 33 additions and 19 deletions
|
@ -15,6 +15,7 @@ import Skeleton from "@saleor/components/Skeleton";
|
||||||
import TableCellAvatar, {
|
import TableCellAvatar, {
|
||||||
AVATAR_MARGIN
|
AVATAR_MARGIN
|
||||||
} from "@saleor/components/TableCellAvatar";
|
} from "@saleor/components/TableCellAvatar";
|
||||||
|
import createNonNegativeValueChangeHandler from "@saleor/utils/handlers/nonNegativeValueChangeHandler";
|
||||||
import React from "react";
|
import React from "react";
|
||||||
import { FormattedMessage } from "react-intl";
|
import { FormattedMessage } from "react-intl";
|
||||||
|
|
||||||
|
@ -164,9 +165,14 @@ const OrderDraftDetailsProducts: React.FC<OrderDraftDetailsProductsProps> = prop
|
||||||
initial={{ quantity: line.quantity }}
|
initial={{ quantity: line.quantity }}
|
||||||
onSubmit={data => onOrderLineChange(line.id, data)}
|
onSubmit={data => onOrderLineChange(line.id, data)}
|
||||||
>
|
>
|
||||||
{({ change, data, hasChanged, submit }) => (
|
{({ change, data, hasChanged, submit }) => {
|
||||||
|
const handleQuantityChange = createNonNegativeValueChangeHandler(
|
||||||
|
change
|
||||||
|
);
|
||||||
|
|
||||||
|
return (
|
||||||
<DebounceForm
|
<DebounceForm
|
||||||
change={change}
|
change={handleQuantityChange}
|
||||||
submit={hasChanged ? submit : undefined}
|
submit={hasChanged ? submit : undefined}
|
||||||
time={200}
|
time={200}
|
||||||
>
|
>
|
||||||
|
@ -179,10 +185,14 @@ const OrderDraftDetailsProducts: React.FC<OrderDraftDetailsProductsProps> = prop
|
||||||
value={data.quantity}
|
value={data.quantity}
|
||||||
onChange={debounce}
|
onChange={debounce}
|
||||||
onBlur={submit}
|
onBlur={submit}
|
||||||
|
inputProps={{
|
||||||
|
min: 1
|
||||||
|
}}
|
||||||
/>
|
/>
|
||||||
)}
|
)}
|
||||||
</DebounceForm>
|
</DebounceForm>
|
||||||
)}
|
);
|
||||||
|
}}
|
||||||
</Form>
|
</Form>
|
||||||
) : (
|
) : (
|
||||||
<Skeleton />
|
<Skeleton />
|
||||||
|
|
|
@ -108894,6 +108894,7 @@ exports[`Storyshots Views / Orders / Order draft default 1`] = `
|
||||||
<input
|
<input
|
||||||
aria-invalid="false"
|
aria-invalid="false"
|
||||||
class="MuiInputBase-input-id MuiOutlinedInput-input-id"
|
class="MuiInputBase-input-id MuiOutlinedInput-input-id"
|
||||||
|
min="1"
|
||||||
name="quantity"
|
name="quantity"
|
||||||
type="number"
|
type="number"
|
||||||
value="2"
|
value="2"
|
||||||
|
@ -108998,6 +108999,7 @@ exports[`Storyshots Views / Orders / Order draft default 1`] = `
|
||||||
<input
|
<input
|
||||||
aria-invalid="false"
|
aria-invalid="false"
|
||||||
class="MuiInputBase-input-id MuiOutlinedInput-input-id"
|
class="MuiInputBase-input-id MuiOutlinedInput-input-id"
|
||||||
|
min="1"
|
||||||
name="quantity"
|
name="quantity"
|
||||||
type="number"
|
type="number"
|
||||||
value="2"
|
value="2"
|
||||||
|
@ -109911,6 +109913,7 @@ exports[`Storyshots Views / Orders / Order draft no user permissions 1`] = `
|
||||||
<input
|
<input
|
||||||
aria-invalid="false"
|
aria-invalid="false"
|
||||||
class="MuiInputBase-input-id MuiOutlinedInput-input-id"
|
class="MuiInputBase-input-id MuiOutlinedInput-input-id"
|
||||||
|
min="1"
|
||||||
name="quantity"
|
name="quantity"
|
||||||
type="number"
|
type="number"
|
||||||
value="2"
|
value="2"
|
||||||
|
@ -110015,6 +110018,7 @@ exports[`Storyshots Views / Orders / Order draft no user permissions 1`] = `
|
||||||
<input
|
<input
|
||||||
aria-invalid="false"
|
aria-invalid="false"
|
||||||
class="MuiInputBase-input-id MuiOutlinedInput-input-id"
|
class="MuiInputBase-input-id MuiOutlinedInput-input-id"
|
||||||
|
min="1"
|
||||||
name="quantity"
|
name="quantity"
|
||||||
type="number"
|
type="number"
|
||||||
value="2"
|
value="2"
|
||||||
|
|
Loading…
Reference in a new issue