Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Externalize all @wordpress packages via DEWP #2056

Merged
merged 31 commits into from
Aug 15, 2023

Conversation

eason9487
Copy link
Member

@eason9487 eason9487 commented Aug 14, 2023

Changes proposed in this Pull Request:

Closes #1833

💡 All code changes have been reviewed and tested in the respective PRs. Therefore, this PR should not require extra rounds of code reviews.

Changelog entry

Dev - Externalize all WordPress JavaScript packages via Dependency Extraction Webpack Plugin (DEWP).

…ent imported from `@wordpress/components`.
… a Modal before.

Currently, the minimum supported version of this extension is WP 5.9.
- This hack doesn't work for WP 5.9 and 6.0.
- WP >= 6.1 itself has already fixed this issue, so it's no longer needed.
…nd-card

Externalize `Flex`, `Card` and their associated group components imported from `@wordpress/components`
…notice-tip

Externalize `Dashicon`, `Icon`, `Notice` and `Tip` components imported from `@wordpress/components`
…ropdown-modal

Externalize `NavigableMenu`, `Dropdown` and `Modal` components imported from `@wordpress/components`
@eason9487 eason9487 self-assigned this Aug 14, 2023
@github-actions github-actions bot added the changelog: dev Developer-facing only change. label Aug 14, 2023
@eason9487 eason9487 changed the title Externalize all @wordpress packages imports via DEWP Externalize all @wordpress packages via DEWP Aug 14, 2023
Externalize all `*Control` components imported from `@wordpress/components`
Opt-in all `@wordpress` packages imports via DEWP
@eason9487 eason9487 marked this pull request as ready for review August 15, 2023 06:56
@eason9487
Copy link
Member Author

I generally tested the externalized components in the following combinations and no issues were found.

  • L-2: WordPress 6.1.3 + WooCommerce 7.8.2
  • L-1: WordPress 6.2.2 + WooCommerce 7.9.0
  • L-0: WordPress 6.3.0 + WooCommerce 8.0.1

I will be merging this PR to complete the entire externalization for all @wordpress packages.

@eason9487 eason9487 merged commit 7b75f5d into develop Aug 15, 2023
@eason9487 eason9487 deleted the dev/externalize-wp-packages branch August 15, 2023 07:04
@eason9487 eason9487 mentioned this pull request Aug 22, 2023
17 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog: dev Developer-facing only change.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Gradually externalize @wordpress/components and finally opt-in all imports via DEWP
1 participant