fix(compat): move babel-polyfill to Give core

[kevinwhoffman]

User Story

As a developer, I want ES6 polyfills included in Give core so that they don't have to be enqueued multiple times in each add-on that requires them, which can result in conflicts like https://github.com/WordImpress/Give-Donation-Upsells-WooCommerce/issues/40.

Currently the WooCommerce and Stripe add-ons are incompatible so we need to take action immediately.

Possible Solution

@mehul0810 and I suggest enqueuing babel-polyfill with its own unique handle in Give core, while removing the polyfill from Stripe and WC add-ons.

Related

• https://github.com/WordImpress/Give-Donation-Upsells-WooCommerce/issues/20
• https://github.com/WordImpress/Give-Donation-Upsells-WooCommerce/issues/40
• https://github.com/WordImpress/Give-Stripe/issues/250

Tasks

• Enqueue babel-polyfill in Give core.
• Remove babel-polyfill from Stripe add-on.
• Remove babel-polyfill from WooCommerce add-on.
• Release patches for Give core, Stripe, and WC simultaneously.

[samsmith89]

When running NPM RUN DEV this throws an error because it isn't present. Is that intended behavior?

[ravinderk]

@samsmith89 I think this issue is with your plugin path. Rename Local Sites to Local-Sites

[raftaar1191]

As this issues being fixed in Give Core version 2.1.7 so removing it from 2.2.0 release