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

fix(react): fix skipRemotes in moduleFederationDevServer #14172

Merged
merged 1 commit into from
Feb 22, 2023

Conversation

Ronaldo-Maciel
Copy link
Contributor

option skipRemotes isn't working when use array multidimensional in moduleFederationConfig

Current Behavior

When using multidimensional array in the remotes option in module-federation-config.js it makes the skipRemotes option not work when putting the names of the applications to be ignored.

Example:

// module-federation.config.js
 const moduleFederationConfig = {
  name: 'admin',
  remotes: [
    ['gallery-admin', 'http://localhost:8000/v1/static/galleries/remoteEntry.js'],
    ['other-admin', 'http://localhost:8000/v1/static/ola/remoteEntry.js'],
  ],
}
// project.json
{ "skipRemotes": ["other-admin"] }

When run the command yarn nx run admin:serve the application in the skip config continues to be triggered

Expected Behavior

When run the command yarn nx run admin:serve the application in the skip config should not be triggered

Related Issue(s)

Fixes #

@vercel
Copy link

vercel bot commented Jan 5, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated
nx-dev ⬜️ Ignored (Inspect) Feb 21, 2023 at 3:03PM (UTC)

@Coly010
Copy link
Contributor

Coly010 commented Jan 10, 2023

Thanks for submitting a PR 🎉 but it might have been worth raising an issue first to see if we could discuss your problem.

can you tell me more about what you tried to do to use skipRemotes that wasn’t working for you?

Module Federation dev Server should already support skipRemotes

@Coly010
Copy link
Contributor

Coly010 commented Jan 10, 2023

Actually, I can see the problem. Let me take a look 😁

@AnerRiskified
Copy link

AnerRiskified commented Feb 21, 2023

any news regarding this issue?
its blocking the possibility of using an external remote

option skipRemotes isn't working when use array multidimensional in moduleFederationConfig
@mandarini mandarini merged commit ba73775 into nrwl:master Feb 22, 2023
@github-actions
Copy link

github-actions bot commented Mar 3, 2023

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 3, 2023
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.

None yet

4 participants