(fix) proxy function props to prevent stale state in callbacks

[imbrian]

context

Callback props (e.g. createOrder, onApprove) are passed to paypal.Buttons in a useEffect, where the SDK Button component is then rendered. While the SDK provides a forceReRender prop to re-invoke the effect, which will update the callback functions passed to the SDK Button, this renders the SDK again (triggering animations, etc). The SDK Button otherwise never receives new functions as the parent component updates, creating stale closures.

change

This PR provides a stable object from which to read the callback functions without requiring an SDK re-render.

resolves #590

[changeset-bot]

🦋 Changeset detected

Latest commit: cf9ed5e

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

This PR includes changesets to release 1 package

Name	Type
@paypal/react-paypal-js	Minor

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

[gregjopa]

Awesome work @imbrian! 💯 I tested the new storybook demo and I see the item quantity updating and it does not cause the buttons to re-render.