Skip to content

Commit

Permalink
fix(react): fix remote generation devServerPort (#18603)
Browse files Browse the repository at this point in the history
  • Loading branch information
nadavshatz committed Aug 21, 2023
1 parent 18ba93a commit 1a0962f
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 2 deletions.
1 change: 0 additions & 1 deletion docs/generated/packages/react/generators/remote.json
Expand Up @@ -152,7 +152,6 @@
"devServerPort": {
"type": "number",
"description": "The port for the dev server of the remote app.",
"default": 4200,
"x-priority": "important"
},
"ssr": {
Expand Down
5 changes: 5 additions & 0 deletions e2e/react-core/src/react-module-federation.test.ts
Expand Up @@ -43,6 +43,11 @@ describe('React Module Federation', () => {
combinedOutput: expect.stringContaining('Test Suites: 1 passed, 1 total'),
});

expect(readPort(shell)).toEqual(4200);
expect(readPort(remote1)).toEqual(4201);
expect(readPort(remote2)).toEqual(4202);
expect(readPort(remote3)).toEqual(4203);

updateFile(
`apps/${shell}/webpack.config.js`,
stripIndents`
Expand Down
1 change: 0 additions & 1 deletion packages/react/src/generators/remote/schema.json
Expand Up @@ -158,7 +158,6 @@
"devServerPort": {
"type": "number",
"description": "The port for the dev server of the remote app.",
"default": 4200,
"x-priority": "important"
},
"ssr": {
Expand Down

1 comment on commit 1a0962f

@vercel
Copy link

@vercel vercel bot commented on 1a0962f Aug 21, 2023

Choose a reason for hiding this comment

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

Successfully deployed to the following URLs:

nx-dev – ./

nx-dev-git-master-nrwl.vercel.app
nx-dev-nrwl.vercel.app
nx-five.vercel.app
nx.dev

Please sign in to comment.