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

feat: async boundary option #1888

Closed
wants to merge 322 commits into from
Closed

Conversation

ScriptedAlchemy
Copy link
Member

Description

add asyncBoundary option to federation plugin interface

This internally enables the automatic async boundary plugin, allowing it to be enabled without manually doing so.

Allows apps to be imported synchronously without getting eager error or call of undefined when not using a async import in the entrypoint.

Related Issue

N/A

Types of changes

  • Docs change / refactoring / dependency upgrade
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

Checklist

  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • I have updated the documentation.

@ScriptedAlchemy ScriptedAlchemy changed the base branch from main to feat_apply_bundler_runtime January 2, 2024 22:56
@@ -553,6 +553,10 @@ export default {
runtime: {
$ref: '#/definitions/EntryRuntime',
},
asyncBoundary: {
Copy link
Member

Choose a reason for hiding this comment

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

Should we update this option in "packages/enhanced/src/schemas/container/ModuleFederationPlugin.check.ts" ?

Copy link
Member Author

Choose a reason for hiding this comment

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

ahh yes, i forgot to do the .ts file only did the schema file.

@zhoushaw zhoushaw changed the title Feat/async boundary option feat: async boundary option Jan 3, 2024
Base automatically changed from feat_apply_bundler_runtime to canary January 4, 2024 02:26
Base automatically changed from canary to main January 8, 2024 12:40
Copy link
Contributor

github-actions bot commented Mar 8, 2024

Stale pull request message

Copy link
Contributor

github-actions bot commented May 8, 2024

Stale pull request message

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants