-
Notifications
You must be signed in to change notification settings - Fork 14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[NO CHANGELOG] Orchestrates Swap/Bridge widgets for Smart Checkout routes #1040
Conversation
…ODS once on app load.
…immutable-sdk into GPR-238/widget-integration
packages/checkout/widgets-lib/src/widgets/sale/hooks/useSignOrder.ts
Outdated
Show resolved
Hide resolved
…-238/widget-integration
...checkout/widgets-lib/src/widgets/sale/components/FundingRouteExecute/FundingRouteExecute.tsx
Show resolved
Hide resolved
packages/checkout/widgets-lib/src/widgets/sale/views/FundWithSmartCheckout.tsx
Outdated
Show resolved
Hide resolved
@@ -27,20 +27,20 @@ export function FundWithSmartCheckout({ subView }: FundWithSmartCheckoutProps) { | |||
|
|||
const { querySmartCheckout, fundingRoutes } = useSaleContext(); | |||
|
|||
let smartCheckoutLoading = false; | |||
const smartCheckoutLoading = useRef(false); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This one is because of useEffect
double rendering, calling SmartCheckout twice in rapid succession causes them both to fail so had to prevent that in dev, not needed in prod though. Is it okay to keep in for now?
Summary
Smart.Checkout.swap.+.execute.flow.mov