Skip to content

[Customer Center] Fix for disabled promo offer button#4142

Merged
vegaro merged 1 commit into
integration/customer_support_workflowfrom
fix-disabled-promo-offer
Aug 2, 2024
Merged

[Customer Center] Fix for disabled promo offer button#4142
vegaro merged 1 commit into
integration/customer_support_workflowfrom
fix-disabled-promo-offer

Conversation

@vegaro

@vegaro vegaro commented Aug 2, 2024

Copy link
Copy Markdown
Member

I noticed the purchase button for the promo offers is always disabled. Changing the order of the modifiers fixes it

@vegaro vegaro requested a review from a team August 2, 2024 11:16
@vegaro vegaro changed the base branch from main to integration/customer_support_workflow August 2, 2024 11:17
promoOfferDetails: promotionalOfferData.promoOfferDetails)
})
.buttonStyle(ManageSubscriptionsButtonStyle())
.disabled(self.viewModel.loadingPath != nil)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm I'm not sure I understand why doing this before actually fixes this... But I'm ok with the fix 👍

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's because it applies it to the sheet as well, and the sheet is the one presenting the promo offer, so basically everything in the sheet is disabled

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ahh makes sense. Thanks for explaining!

@vegaro vegaro merged commit dbd43aa into integration/customer_support_workflow Aug 2, 2024
@vegaro vegaro deleted the fix-disabled-promo-offer branch August 2, 2024 13:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants