Skip to content

Commit 0fce03f

Browse files
authored
remove phone number input tooltip (#16075)
1 parent 804e1c5 commit 0fce03f

File tree

2 files changed

+3
-5
lines changed

2 files changed

+3
-5
lines changed

src/Apps/Order/Routes/Shipping/Components/FulfillmentDetailsForm.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -553,7 +553,6 @@ const FulfillmentDetailsFormLayout = (
553553
name="attributes.phoneNumber"
554554
title="Phone number"
555555
type="tel"
556-
description={"Required for pickup logistics"}
557556
placeholder="Add phone number including country code"
558557
pattern="[^a-z]+"
559558
value={values.attributes.phoneNumber}
@@ -564,6 +563,7 @@ const FulfillmentDetailsFormLayout = (
564563
errors.attributes?.phoneNumber
565564
}
566565
data-testid="AddressForm_pickupPhoneNumber"
566+
required
567567
/>
568568
<Spacer y={4} />
569569
</>

src/Apps/Order/Routes/Shipping/Components/__tests__/FulfillmentDetailsForm.jest.tsx

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ import {
77
within,
88
} from "@testing-library/react"
99
import userEvent from "@testing-library/user-event"
10+
import { useFlag } from "@unleash/proxy-client-react"
1011
import {
1112
FulfillmentDetailsForm,
1213
type FulfillmentDetailsFormProps,
@@ -18,7 +19,6 @@ import {
1819
} from "Apps/Order/Routes/Shipping/Utils/shippingUtils"
1920
import { fillAddressForm } from "Components/__tests__/Utils/addressForm2"
2021
import { flushPromiseQueue } from "DevTools/flushPromiseQueue"
21-
import { useFlag } from "@unleash/proxy-client-react"
2222
import type { DeepPartial } from "Utils/typeSupport"
2323
import { useTracking } from "react-tracking"
2424

@@ -160,9 +160,7 @@ describe("FulfillmentDetailsForm", () => {
160160
)
161161

162162
expect(phoneNumberField).toBeVisible()
163-
expect(
164-
screen.getByText("Required for pickup logistics"),
165-
).toBeInTheDocument()
163+
expect(screen.getByText("*Required")).toBeInTheDocument()
166164
})
167165

168166
it("tries to go back to fulfillment details when the user clicks pickup", async () => {

0 commit comments

Comments
 (0)