fix: Do not refund payments when cancelling bookings #20459
Triggered via pull request
March 13, 2024 20:07
joeauyeung
synchronize
#13924
Status
Success
Total duration
15s
Artifacts
–
semantic-pull-requests.yml
on: pull_request_target
Validate PR title
5s
Annotations
58 warnings
apps/web/playwright/teams.e2e.ts#L22
[playwright/no-skipped-test] Unexpected use of the `.skip()` annotation.
|
apps/web/playwright/teams.e2e.ts#L157
[@typescript-eslint/no-non-null-assertion] Forbidden non-null assertion.
|
apps/web/playwright/teams.e2e.ts#L162
[playwright/no-skipped-test] Unexpected use of the `.skip()` annotation.
|
packages/app-store/_utils/useAddAppMutation.ts#L30
[@typescript-eslint/no-unused-vars] 'useAddAppMutationVariables' is defined but never used. Allowed unused vars must match /^_/u.
|
apps/web/playwright/teams.e2e.ts#L197
[playwright/no-skipped-test] Unexpected use of the `.skip()` annotation.
|
apps/web/playwright/teams.e2e.ts#L200
[@typescript-eslint/no-non-null-assertion] Forbidden non-null assertion.
|
packages/app-store/googlecalendar/tests/google-calendar.e2e.ts#L25
[playwright/no-skipped-test] Unexpected use of the `.skip()` annotation.
|
apps/web/test/lib/team-event-types.test.ts#L79
[@typescript-eslint/no-unused-vars] 'test' is assigned a value but never used. Allowed unused vars must match /^_/u.
|
packages/app-store/googlecalendar/tests/google-calendar.e2e.ts#L38
[playwright/no-skipped-test] Unexpected use of the `.skip()` annotation.
|
apps/web/test/utils/bookingScenario/expects.ts#L173
[@typescript-eslint/no-non-null-assertion] Forbidden non-null assertion.
|
packages/app-store/googlecalendar/tests/google-calendar.e2e.ts#L41
[playwright/no-skipped-test] Unexpected use of the `.skip()` annotation.
|
apps/web/test/utils/bookingScenario/expects.ts#L738
[@typescript-eslint/no-unused-vars] 'organizer' is defined but never used. Allowed unused args must match /^_/u.
|
packages/app-store/googlecalendar/tests/google-calendar.e2e.ts#L42
[playwright/no-skipped-test] Unexpected use of the `.skip()` annotation.
|
packages/app-store/googlecalendar/tests/google-calendar.e2e.ts#L60
[playwright/no-skipped-test] Unexpected use of the `.skip()` annotation.
|
packages/app-store/googlecalendar/tests/google-calendar.e2e.ts#L72
[playwright/no-skipped-test] Unexpected use of the `.skip()` annotation.
|
packages/app-store/googlecalendar/tests/google-calendar.e2e.ts#L77
[playwright/no-skipped-test] Unexpected use of the `.skip()` annotation.
|
packages/app-store/googlecalendar/tests/google-calendar.e2e.ts#L104
[playwright/no-skipped-test] Unexpected use of the `.skip()` annotation.
|
packages/app-store/googlecalendar/tests/testUtils.ts#L110
[@typescript-eslint/no-explicit-any] Unexpected any. Specify a different type.
|
packages/app-store/jelly/lib/VideoApiAdapter.ts#L11
[@typescript-eslint/no-unused-vars] 'event' is defined but never used. Allowed unused args must match /^_/u.
|
packages/app-store/jelly/lib/VideoApiAdapter.ts#L32
[@typescript-eslint/no-unused-vars] 'event' is defined but never used. Allowed unused args must match /^_/u.
|
packages/app-store/jelly/lib/VideoApiAdapter.ts#L41
[@typescript-eslint/no-unused-vars] 'uid' is defined but never used. Allowed unused args must match /^_/u.
|
packages/app-store/jelly/lib/VideoApiAdapter.ts#L45
[@typescript-eslint/no-unused-vars] 'dateFrom' is defined but never used. Allowed unused args must match /^_/u.
|
packages/app-store/jelly/lib/VideoApiAdapter.ts#L45
[@typescript-eslint/no-unused-vars] 'dateTo' is defined but never used. Allowed unused args must match /^_/u.
|
packages/app-store/routing-forms/components/getServerSidePropsSingleForm.ts#L92
[@typescript-eslint/no-unused-vars] 'u' is assigned a value but never used. Allowed unused vars must match /^_/u.
|
packages/app-store/routing-forms/pages/route-builder/[...appPages].tsx#L149
[react-hooks/exhaustive-deps] React Hook useEffect has missing dependencies: 'eventOptions' and 'route'. Either include them or remove the dependency array. If 'setCustomEventTypeSlug' needs the current value of 'route', you can also switch to useReducer instead of useState and read 'route' in the reducer.
|
packages/ui/components/image-uploader/BannerUploader.tsx#L130
[react-hooks/exhaustive-deps] React Hook useEffect has missing dependencies: 'height', 't', and 'width'. Either include them or remove the dependency array.
|
apps/web/app/WithAppDirSsg.tsx#L5
[@typescript-eslint/no-explicit-any] Unexpected any. Specify a different type.
|
apps/web/app/WithAppDirSsr.tsx#L5
[@typescript-eslint/no-explicit-any] Unexpected any. Specify a different type.
|
apps/web/app/WithEmbedSSR.tsx#L12
[@typescript-eslint/no-explicit-any] Unexpected any. Specify a different type.
|
apps/web/app/future/apps/[slug]/[...pages]/page.tsx#L130
[@typescript-eslint/no-non-null-assertion] Forbidden non-null assertion.
|
apps/web/app/layoutHOC.tsx#L9
[@typescript-eslint/no-explicit-any] Unexpected any. Specify a different type.
|
apps/web/app/layoutHOC.tsx#L16
[@typescript-eslint/no-explicit-any] Unexpected any. Specify a different type.
|
apps/web/components/eventtype/EventAvailabilityTab.tsx#L227
[react-hooks/exhaustive-deps] React Hook useEffect has missing dependencies: 'eventType.schedule', 'eventType.scheduleName', 'getValues', 'isChildrenManagedEventType', 'isManagedEventType', 'setValue', 't', and 'watchSchedule'. Either include them or remove the dependency array.
|
apps/web/components/eventtype/EventTeamTab.tsx#L152
[@typescript-eslint/no-unused-vars] 'helperText' is defined but never used. Allowed unused args must match /^_/u.
|
apps/web/components/eventtype/EventTeamTab.tsx#L453
[@typescript-eslint/no-unused-vars] 't' is assigned a value but never used. Allowed unused vars must match /^_/u.
|
apps/web/components/eventtype/EventTypeSingleLayout.tsx#L286
[react-hooks/exhaustive-deps] React Hook useMemo has missing dependencies: 'eventType.id' and 'formMethods'. Either include them or remove the dependency array.
|
apps/web/components/eventtype/EventTypeSingleLayout.tsx#L297
[react-hooks/exhaustive-deps] React Hook useMemo has a complex expression in the dependency array. Extract it to a separate variable so it can be statically checked.
|
apps/web/components/settings/SecondaryEmailModal.tsx#L49
[react-hooks/exhaustive-deps] React Hook useEffect has missing dependencies: 'clearErrorMessage' and 'formMethods'. Either include them or remove the dependency array. If 'clearErrorMessage' changes too often, find the parent component that defines it and wrap that definition in useCallback.
|
apps/web/lib/apps/[slug]/getStaticProps.tsx#L13
[@typescript-eslint/no-unused-vars] 'md' is assigned a value but never used. Allowed unused vars must match /^_/u.
|
apps/web/lib/team/[slug]/[type]/getServerSideProps.tsx#L26
[@typescript-eslint/no-unused-vars] 'queryDuration' is assigned a value but never used. Allowed unused vars must match /^_/u.
|
apps/web/middleware.ts#L14
[@typescript-eslint/no-explicit-any] Unexpected any. Specify a different type.
|
apps/web/pages/settings/my-account/calendars.tsx#L13
[@typescript-eslint/no-unused-vars] 'SkeletonLoader' is assigned a value but never used. Allowed unused vars must match /^_/u.
|
apps/web/pages/settings/my-account/profile.tsx#L609
[@typescript-eslint/no-unused-vars] 'organization' is assigned a value but never used. Allowed unused vars must match /^_/u.
|
apps/web/playwright/ab-tests-redirect.e2e.ts#L27
[playwright/no-conditional-in-test] Avoid having conditionals in tests
|
apps/web/playwright/ab-tests-redirect.e2e.ts#L43
[playwright/no-conditional-in-test] Avoid having conditionals in tests
|
apps/web/playwright/ab-tests-redirect.e2e.ts#L59
[playwright/no-conditional-in-test] Avoid having conditionals in tests
|
apps/web/playwright/ab-tests-redirect.e2e.ts#L75
[playwright/no-conditional-in-test] Avoid having conditionals in tests
|
apps/web/playwright/ab-tests-redirect.e2e.ts#L91
[playwright/no-conditional-in-test] Avoid having conditionals in tests
|
apps/web/playwright/ab-tests-redirect.e2e.ts#L107
[playwright/no-conditional-in-test] Avoid having conditionals in tests
|
apps/web/playwright/ab-tests-redirect.e2e.ts#L123
[playwright/no-conditional-in-test] Avoid having conditionals in tests
|
apps/web/playwright/app-store.e2e.ts#L13
[playwright/no-skipped-test] Unexpected use of the `.skip()` annotation.
|
apps/web/playwright/organization/booking.e2e.ts#L349
[playwright/no-conditional-in-test] Avoid having conditionals in tests
|
apps/web/playwright/payment-apps.e2e.ts#L230
[playwright/no-conditional-in-test] Avoid having conditionals in tests
|
apps/web/playwright/payment-apps.e2e.ts#L274
[playwright/no-conditional-in-test] Avoid having conditionals in tests
|
apps/web/playwright/payment.e2e.ts#L17
[playwright/no-skipped-test] Unexpected use of the `.skip()` annotation.
|
apps/web/playwright/payment.e2e.ts#L18
[playwright/no-skipped-test] Unexpected use of the `.skip()` annotation.
|
apps/web/playwright/profile.e2e.ts#L389
[playwright/no-conditional-in-test] Avoid having conditionals in tests
|
apps/web/playwright/signup.e2e.ts#L193
[@typescript-eslint/no-non-null-assertion] Forbidden non-null assertion.
|