Skip to content
This repository was archived by the owner on Jul 12, 2024. It is now read-only.

Conversation

joshuatf
Copy link
Contributor

@joshuatf joshuatf commented Jul 22, 2021

Fixes #7402

Fixes the SlotFill due to a broken webpack build. Resolved by manually importing createElement.

Huge shoutout to @louwie17 for getting to the bottom of this.

Screenshots

Screen Shot 2021-07-22 at 1 19 44 PM

Detailed test instructions:

  1. Install and build this WC Paypal plugin Add quick setup methods for recommended payment gateways woocommerce-paypal-payments#164
  2. Run yarn link inside packages/onboarding in WooCommerce Admin.
  3. In the modules/ppcp-onboarding folder in PayPal, run yarn link @woocommerce/onboarding && yarn build
  4. Set your store to any country besides India and don't select "CBD" as an industry during onboarding.
  5. Navigate to the payment gateways task
  6. Go through the PayPal gateway task.
  7. Note that the PayPal configure step loads with a connection button or form shown above.

@joshuatf joshuatf requested a review from a team July 22, 2021 17:31
@joshuatf joshuatf self-assigned this Jul 22, 2021
Copy link
Contributor

@louwie17 louwie17 left a comment

Choose a reason for hiding this comment

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

LGTM 🚀 had already tested this from our chat on Slack, looks good to me 👍
It does seem like createElement has been imported in all the other JSX components within this package.

@joshuatf joshuatf force-pushed the fix/payment-gateway-slot-fill branch from 5746d80 to e73bee7 Compare July 23, 2021 13:01
@joshuatf joshuatf merged commit ef5df46 into main Jul 23, 2021
@joshuatf joshuatf deleted the fix/payment-gateway-slot-fill branch July 23, 2021 13:35
octaedro pushed a commit that referenced this pull request Jul 26, 2021
* Import createElement to fix build issues with SlotFill

* Add changelog entry
ObliviousHarmony pushed a commit to woocommerce/woocommerce that referenced this pull request Mar 18, 2022
…oocommerce-admin#7403)

* Import createElement to fix build issues with SlotFill

* Add changelog entry
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Payment SlotFill does not get filled by registered plugin

3 participants