Skip to content

Conversation

@EvanReinstein
Copy link
Contributor

No description provided.

@EvanReinstein EvanReinstein requested a review from a team as a code owner December 1, 2025 20:14
@changeset-bot
Copy link

changeset-bot bot commented Dec 1, 2025

🦋 Changeset detected

Latest commit: 37b66a4

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@paypal/paypal-js Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

};

export type PayPalGuestPresentationModeOptions =
PresentationModeOptionsForAuto & {
Copy link
Contributor

Choose a reason for hiding this comment

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

Where we going to omit presentationMode here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh, I thought we discussed using Omit in the BCDC hook itself and leaving this type unchanged. Something like this, since we want to remove presentationMode from the hook's arguments but still be able to correctly type the option we pass to .start.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Discussed with @kand and we are good on this implementation as is currently.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants