search

Parsely / wp-parsely

The official WordPress plugin for Parse.ly - makes it a snap to add the required tracking code to enable Parse.ly on your WordPress site.
https://wordpress.org/plugins/wp-parsely/
GNU General Public License v2.0
62 stars 31 forks source link

Consider polyfill solution to allow wp-scripts upgrades #2624

Open acicovic opened 1 month ago

acicovic commented 1 month ago

Is your feature request related to a problem?

When we attempted to upgrade to wp-scripts v28, our tooling based on React disappeared. This was experienced by others as well, and the main issue is here.

Apparently, there are 2 propositions:

  1. Don't upgrade to wp-scripts v28 until the minimum WP version for the plugin is 6.6. This is what's officially proposed in the changelog and related article.
  2. There's a mention of using a polyfill solution.

Describe the solution you'd like

For now, we've paused dependabot updates for v28.

Although this is what's officially proposed, the first proposition's viability isn't a given in my opinion. If a polyfill solution is implementable, we should look into this possibility. It seems that the polyfill solution will require updating our minimum WP version to at least 5.9, which shouldn't be an issue.

acicovic commented 1 month ago

Let's remember to remove "@typescript-eslint/method-signature-style": "error" from .eslintrc when we're done with this.