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
feat: dynamic messages - modal instrumentation #2380
base: main
Are you sure you want to change the base?
Conversation
src/zoid/buttons/component.jsx
Outdated
const modalInstance = await getModal( | ||
clientID, | ||
merchantID, | ||
buttonSessionID | ||
); | ||
return modalInstance?.show({ | ||
amount, | ||
offer: offerType, | ||
buttonSessionId: buttonSessionID, |
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.
We should only need to pass the buttonSessionID
into either getModal
or show
. I don't think we need to do both. Since we need it in getModal
for the events, we can probably omit it from show
since the buttonSessionID
value shouldn't change.
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.
okay i will remove one from the show right now
Description
Why are we making these changes? Include references to any related Jira tasks or GitHub Issues
Reproduction Steps (if applicable)
Screenshots (if applicable)
Dependent Changes (if applicable)
Groups who should review (if applicable)
❤️ Thank you!