Fix missing metadata and payment balance on unconfirmed orders (#2314)
* Fix missing metadata and payment balance on unconfirmed orders * Update changelog with fix missing fields on order page Co-authored-by: Patryk Andrzejewski <vox3r69@gmail.com>
This commit is contained in:
parent
32c2ff5f13
commit
214b677684
4 changed files with 50 additions and 49 deletions
|
@ -6,6 +6,7 @@ All notable, unreleased changes to this project will be documented in this file.
|
||||||
|
|
||||||
- Pass query params in `ORDER_DETAILS_MORE_ACTIONS` and `PRODUCT_DETAILS_MORE_ACTIONS` mounting points - #2100 by @witoszekdev
|
- Pass query params in `ORDER_DETAILS_MORE_ACTIONS` and `PRODUCT_DETAILS_MORE_ACTIONS` mounting points - #2100 by @witoszekdev
|
||||||
- Add product variant reference attribute - #2268 by @droniu
|
- Add product variant reference attribute - #2268 by @droniu
|
||||||
|
- Fix missing metadata and payment balance on unconfirmed orders - #2314 by @orzechdev
|
||||||
- Fix exit form dialog false positive - #2311 by @orzechdev
|
- Fix exit form dialog false positive - #2311 by @orzechdev
|
||||||
- Fix no product error on unconfirmed order lines - #2324 by @orzechdev
|
- Fix no product error on unconfirmed order lines - #2324 by @orzechdev
|
||||||
|
|
||||||
|
|
|
@ -24,13 +24,13 @@ import {
|
||||||
import { SubmitPromise } from "@saleor/hooks/useForm";
|
import { SubmitPromise } from "@saleor/hooks/useForm";
|
||||||
import useNavigator from "@saleor/hooks/useNavigator";
|
import useNavigator from "@saleor/hooks/useNavigator";
|
||||||
import { sectionNames } from "@saleor/intl";
|
import { sectionNames } from "@saleor/intl";
|
||||||
import { ConfirmButtonTransitionState, makeStyles } from "@saleor/macaw-ui";
|
import { ConfirmButtonTransitionState } from "@saleor/macaw-ui";
|
||||||
import OrderChannelSectionCard from "@saleor/orders/components/OrderChannelSectionCard";
|
import OrderChannelSectionCard from "@saleor/orders/components/OrderChannelSectionCard";
|
||||||
import { orderListUrl } from "@saleor/orders/urls";
|
import { orderListUrl } from "@saleor/orders/urls";
|
||||||
import { mapMetadataItemToInput } from "@saleor/utils/maps";
|
import { mapMetadataItemToInput } from "@saleor/utils/maps";
|
||||||
import useMetadataChangeTrigger from "@saleor/utils/metadata/useMetadataChangeTrigger";
|
import useMetadataChangeTrigger from "@saleor/utils/metadata/useMetadataChangeTrigger";
|
||||||
import React from "react";
|
import React from "react";
|
||||||
import { defineMessages, useIntl } from "react-intl";
|
import { useIntl } from "react-intl";
|
||||||
|
|
||||||
import { getMutationErrors, maybe } from "../../../misc";
|
import { getMutationErrors, maybe } from "../../../misc";
|
||||||
import OrderCustomer from "../OrderCustomer";
|
import OrderCustomer from "../OrderCustomer";
|
||||||
|
@ -42,25 +42,11 @@ import OrderHistory, { FormData as HistoryFormData } from "../OrderHistory";
|
||||||
import OrderInvoiceList from "../OrderInvoiceList";
|
import OrderInvoiceList from "../OrderInvoiceList";
|
||||||
import OrderPayment from "../OrderPayment/OrderPayment";
|
import OrderPayment from "../OrderPayment/OrderPayment";
|
||||||
import OrderUnfulfilledProductsCard from "../OrderUnfulfilledProductsCard";
|
import OrderUnfulfilledProductsCard from "../OrderUnfulfilledProductsCard";
|
||||||
|
import { messages } from "./messages";
|
||||||
|
import { useStyles } from "./styles";
|
||||||
import Title from "./Title";
|
import Title from "./Title";
|
||||||
import { filteredConditionalItems, hasAnyItemsReplaceable } from "./utils";
|
import { filteredConditionalItems, hasAnyItemsReplaceable } from "./utils";
|
||||||
|
|
||||||
const useStyles = makeStyles(
|
|
||||||
theme => ({
|
|
||||||
date: {
|
|
||||||
marginBottom: theme.spacing(3),
|
|
||||||
},
|
|
||||||
header: {
|
|
||||||
display: "flex",
|
|
||||||
justifyContent: "space-between",
|
|
||||||
marginBottom: 0,
|
|
||||||
},
|
|
||||||
}),
|
|
||||||
{
|
|
||||||
name: "OrderDetailsPage",
|
|
||||||
},
|
|
||||||
);
|
|
||||||
|
|
||||||
export interface OrderDetailsPageProps {
|
export interface OrderDetailsPageProps {
|
||||||
order: OrderDetailsFragment;
|
order: OrderDetailsFragment;
|
||||||
shop: OrderDetailsQuery["shop"];
|
shop: OrderDetailsQuery["shop"];
|
||||||
|
@ -99,24 +85,6 @@ export interface OrderDetailsPageProps {
|
||||||
onSubmit(data: MetadataFormData): SubmitPromise;
|
onSubmit(data: MetadataFormData): SubmitPromise;
|
||||||
}
|
}
|
||||||
|
|
||||||
const messages = defineMessages({
|
|
||||||
cancelOrder: {
|
|
||||||
id: "9ZtJhn",
|
|
||||||
defaultMessage: "Cancel order",
|
|
||||||
description: "cancel button",
|
|
||||||
},
|
|
||||||
confirmOrder: {
|
|
||||||
id: "maxT+q",
|
|
||||||
defaultMessage: "Confirm order",
|
|
||||||
description: "save button",
|
|
||||||
},
|
|
||||||
returnOrder: {
|
|
||||||
id: "+RjQjs",
|
|
||||||
defaultMessage: "Return / Replace order",
|
|
||||||
description: "return button",
|
|
||||||
},
|
|
||||||
});
|
|
||||||
|
|
||||||
const OrderDetailsPage: React.FC<OrderDetailsPageProps> = props => {
|
const OrderDetailsPage: React.FC<OrderDetailsPageProps> = props => {
|
||||||
const {
|
const {
|
||||||
disabled,
|
disabled,
|
||||||
|
@ -299,8 +267,6 @@ const OrderDetailsPage: React.FC<OrderDetailsPageProps> = props => {
|
||||||
/>
|
/>
|
||||||
</React.Fragment>
|
</React.Fragment>
|
||||||
))}
|
))}
|
||||||
{!isOrderUnconfirmed && (
|
|
||||||
<>
|
|
||||||
<OrderPayment
|
<OrderPayment
|
||||||
order={order}
|
order={order}
|
||||||
onCapture={onPaymentCapture}
|
onCapture={onPaymentCapture}
|
||||||
|
@ -310,8 +276,6 @@ const OrderDetailsPage: React.FC<OrderDetailsPageProps> = props => {
|
||||||
/>
|
/>
|
||||||
<CardSpacer />
|
<CardSpacer />
|
||||||
<Metadata data={data} onChange={changeMetadata} />
|
<Metadata data={data} onChange={changeMetadata} />
|
||||||
</>
|
|
||||||
)}
|
|
||||||
<OrderHistory
|
<OrderHistory
|
||||||
history={order?.events}
|
history={order?.events}
|
||||||
orderCurrency={order?.total?.gross.currency}
|
orderCurrency={order?.total?.gross.currency}
|
||||||
|
|
19
src/orders/components/OrderDetailsPage/messages.ts
Normal file
19
src/orders/components/OrderDetailsPage/messages.ts
Normal file
|
@ -0,0 +1,19 @@
|
||||||
|
import { defineMessages } from "react-intl";
|
||||||
|
|
||||||
|
export const messages = defineMessages({
|
||||||
|
cancelOrder: {
|
||||||
|
id: "9ZtJhn",
|
||||||
|
defaultMessage: "Cancel order",
|
||||||
|
description: "cancel button",
|
||||||
|
},
|
||||||
|
confirmOrder: {
|
||||||
|
id: "maxT+q",
|
||||||
|
defaultMessage: "Confirm order",
|
||||||
|
description: "save button",
|
||||||
|
},
|
||||||
|
returnOrder: {
|
||||||
|
id: "+RjQjs",
|
||||||
|
defaultMessage: "Return / Replace order",
|
||||||
|
description: "return button",
|
||||||
|
},
|
||||||
|
});
|
17
src/orders/components/OrderDetailsPage/styles.ts
Normal file
17
src/orders/components/OrderDetailsPage/styles.ts
Normal file
|
@ -0,0 +1,17 @@
|
||||||
|
import { makeStyles } from "@saleor/macaw-ui";
|
||||||
|
|
||||||
|
export const useStyles = makeStyles(
|
||||||
|
theme => ({
|
||||||
|
date: {
|
||||||
|
marginBottom: theme.spacing(3),
|
||||||
|
},
|
||||||
|
header: {
|
||||||
|
display: "flex",
|
||||||
|
justifyContent: "space-between",
|
||||||
|
marginBottom: 0,
|
||||||
|
},
|
||||||
|
}),
|
||||||
|
{
|
||||||
|
name: "OrderDetailsPage",
|
||||||
|
},
|
||||||
|
);
|
Loading…
Reference in a new issue