-
Notifications
You must be signed in to change notification settings - Fork 307
Add full stack Node.js solution #736
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
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This pull request introduces a full stack Node.js solution integrated with a .NET Aspire host to demonstrate an end-to-end JavaScript solution with a Node.js API and a React frontend.
- Adds sample documentation describing the architecture and prerequisites.
- Configures the .NET Aspire host to run both the Node.js API and React app.
- Implements a simple Express API returning random weather forecast data and a corresponding React app that consumes it.
Reviewed Changes
Copilot reviewed 25 out of 25 changed files in this pull request and generated 3 comments.
Show a summary per file
File | Description |
---|---|
samples/FullStackJS/README.md | Documentation introducing the full stack solution and its usage |
samples/FullStackJS/FullStackJS.AppHost/Program.cs | Configuration of the .NET Aspire host and integration of Node apps |
samples/FullStackJS/api/index.ts | Express API serving random weather data |
samples/FullStackJS/app/webpack.config.js | Webpack configuration for the React app with proxy settings |
samples/FullStackJS/app/src/index.js | Entry point for the React app |
samples/FullStackJS/app/src/components/App.js | React component fetching and displaying weather forecast data |
Co-authored-by: Copilot <[email protected]>
@@ -0,0 +1,22 @@ | |||
FROM node:23-alpine as build |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should look into optimizations we can do with node docker files. They are many.
Co-authored-by: David Fowler <[email protected]>
Add full stack Node.js solution, this PR:
api
express-based Node.js app that serves random weather forecast data (much like the ASP.NET Core Minimal API template).app
that's copied from the existing React bits in the JavaScript samples.Contributes to dotnet/docs-aspire#2767