Fix "Edit Booking" Icon (#2536)

pull/2544/head
Hariom Balhara 2022-04-18 15:55:56 +05:30 committed by GitHub
parent 7fd57b88dc
commit 102ca5403d
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
4 changed files with 2450 additions and 821 deletions

View File

@ -1,4 +1,4 @@
import { BanIcon, CheckIcon, ClockIcon, XIcon } from "@heroicons/react/outline"; import { BanIcon, CheckIcon, ClockIcon, XIcon, PencilAltIcon } from "@heroicons/react/outline";
import { PaperAirplaneIcon } from "@heroicons/react/outline"; import { PaperAirplaneIcon } from "@heroicons/react/outline";
import { BookingStatus } from "@prisma/client"; import { BookingStatus } from "@prisma/client";
import dayjs from "dayjs"; import dayjs from "dayjs";
@ -87,7 +87,7 @@ function BookingListItem(booking: BookingItem) {
actions: [ actions: [
{ {
id: "edit", id: "edit",
icon: ClockIcon, icon: PencilAltIcon,
label: t("edit_booking"), label: t("edit_booking"),
href: `/reschedule/${booking.uid}`, href: `/reschedule/${booking.uid}`,
}, },

View File

@ -213,7 +213,7 @@ const AvailabilityPage = ({ profile, plan, eventType, workingHours, previousPage
<div> <div>
<p <p
className="mt-8 mb-2 text-gray-600 dark:text-white" className="mt-8 mb-2 text-gray-600 dark:text-white"
data-testid="former_time_p"> data-testid="former_time_p_mobile">
{t("former_time")} {t("former_time")}
</p> </p>
<p className="text-gray-500 line-through dark:text-white"> <p className="text-gray-500 line-through dark:text-white">
@ -230,7 +230,6 @@ const AvailabilityPage = ({ profile, plan, eventType, workingHours, previousPage
</div> </div>
<div className="px-4 sm:flex sm:p-4 sm:py-5"> <div className="px-4 sm:flex sm:p-4 sm:py-5">
{/* Following div is marked hidden. Should we remove it ? */}
<div <div
className={ className={
"hidden pr-8 sm:border-r sm:dark:border-gray-700 md:flex md:flex-col " + "hidden pr-8 sm:border-r sm:dark:border-gray-700 md:flex md:flex-col " +
@ -292,7 +291,9 @@ const AvailabilityPage = ({ profile, plan, eventType, workingHours, previousPage
)} )}
{booking?.startTime && rescheduleUid && ( {booking?.startTime && rescheduleUid && (
<div> <div>
<p className="mt-4 mb-2 text-gray-600 dark:text-white" data-testid="former_time_p"> <p
className="mt-4 mb-2 text-gray-600 dark:text-white"
data-testid="former_time_p_desktop">
{t("former_time")} {t("former_time")}
</p> </p>
<p className="text-gray-500 line-through dark:text-white"> <p className="text-gray-500 line-through dark:text-white">

View File

@ -104,7 +104,7 @@ test.describe("Reschedule Tests", async () => {
await page.goto( await page.goto(
`/${originalBooking?.user?.username}/${eventType?.slug}?rescheduleUid=${originalBooking?.uid}` `/${originalBooking?.user?.username}/${eventType?.slug}?rescheduleUid=${originalBooking?.uid}`
); );
const formerTimeElement = await page.locator('[data-testid="former_time_p"]'); const formerTimeElement = await page.locator('[data-testid="former_time_p_desktop"]');
await expect(formerTimeElement).toBeVisible(); await expect(formerTimeElement).toBeVisible();
}); });

3258
yarn.lock

File diff suppressed because it is too large Load Diff