-
Notifications
You must be signed in to change notification settings - Fork 16
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] feat: primary revenue widget hooks - part 2 #900
Conversation
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.
Looks good 🔥
packages/checkout/widgets-lib/src/widgets/primary-revenue/hooks/useSignOrder.ts
Show resolved
Hide resolved
@@ -35,7 +36,15 @@ export function PrimaryRevenueWidget(props: PrimaryRevenueWidgetProps) { | |||
config, amount, items, fromContractAddress, env, environmentId, | |||
} = props; | |||
|
|||
console.log('@@@ PrimaryRevenueWidget', config, amount, items, fromContractAddress, env, environmentId); | |||
console.log( |
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.
keeping it to skip eslint check, gets remove in the next PRs.
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.
You can add an eslint ignore above the console log
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.
thanks.
packages/checkout/widgets-lib/src/widgets/primary-revenue/hooks/useSignOrder.ts
Show resolved
Hide resolved
@ZacharyCouchman / @proletesseract May please request your review here. Cheers |
}; | ||
|
||
// eslint-disable-next-line @typescript-eslint/naming-convention | ||
const SharedContext = createContext<SharedContextValues>({ |
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.
Just a question on naming, any reason why you are not calling this PrimaryRevenueContext? Or something like that?
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.
No reason, we can name to PrimaryRevenueContext
as per your suggestion.
Summary
Add a shared context provider for primary revenue views
Why the changes
Enables sharing the state the views and subviews required for smart checkouts and sign order