Skip to content

Commit e809b98

Browse files
Merge pull request #19449 from mozilla/PAY-3260-cancel-subscription-button-is-active-without-marking-checkbox
fix(payments-next): “Cancel subscription” button is active without marking the checkbox for users that had their subscription previously reactivated
2 parents 6276bf7 + 27e372c commit e809b98

1 file changed

Lines changed: 9 additions & 2 deletions

File tree

  • libs/payments/ui/src/lib/client/components/SubscriptionContent

libs/payments/ui/src/lib/client/components/SubscriptionContent/index.tsx

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -118,6 +118,7 @@ export const SubscriptionContent = ({
118118
if (result.ok) {
119119
setOpenCancellationDialog(true);
120120
setShowCancel(false);
121+
setCheckedState(false);
121122
setShowCancelActionError(false);
122123
} else {
123124
setShowCancelActionError(true);
@@ -463,7 +464,10 @@ export const SubscriptionContent = ({
463464
<SubmitButton
464465
className="h-10 w-full tablet:w-1/2"
465466
variant={ButtonVariant.Primary}
466-
onClick={() => setShowCancel(false)}
467+
onClick={() => {
468+
setCheckedState(false);
469+
setShowCancel(false);
470+
}}
467471
aria-label={`Stay subscribed to ${productName}`}
468472
showLoadingSpinner={false}
469473
>
@@ -637,7 +641,10 @@ export const SubscriptionContent = ({
637641
<SubmitButton
638642
className="h-10"
639643
variant={ButtonVariant.Secondary}
640-
onClick={() => setShowCancel(true)}
644+
onClick={() => {
645+
setCheckedState(false);
646+
setShowCancel(true);
647+
}}
641648
aria-label={`Cancel your subscription for ${productName}`}
642649
>
643650
Cancel

0 commit comments

Comments
 (0)