-
-
Notifications
You must be signed in to change notification settings - Fork 14
Rebuild application with Node.js Express in /new folder #37
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: master
Are you sure you want to change the base?
Conversation
Co-authored-by: sayedimac <25403967+sayedimac@users.noreply.github.com>
Co-authored-by: sayedimac <25403967+sayedimac@users.noreply.github.com>
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 migrates the ASP.NET Core MVC Slots application to a Node.js Express implementation while preserving original functionality. Key changes include a complete restructuring under a new /new folder with an Express server, conversion of Razor views to EJS templates, and configuration adjustments using environment variables.
Reviewed Changes
Copilot reviewed 60 out of 60 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| obj/Debug/... | Added auto-generated scoped CSS files for styling. |
| new/app.js | Implements the Express server with routes and error handling. |
| new/package.json | Configured project dependencies and scripts for Node.js. |
| new/views/**/*.ejs | Converted Razor views to EJS templates reflecting original pages. |
| new/public/** | Includes static assets and third-party libraries. |
| new/README.md, new/.gitignore | Updated documentation and ignore rules for the new project. |
| # TernJS port file | ||
| .tern-port | ||
|
|
||
| # Stores VSCode versions used for testing VSCode extensions |
Copilot
AI
Jun 6, 2025
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.
Consider adding the '/obj' directory to .gitignore to avoid tracking build artifacts generated during the build process.
| @@ -0,0 +1,5 @@ | |||
| <div class="text-center"> | |||
| Contact us about... pretty much Anything Azure related - or Rugby World Cup related ;) | |||
Copilot
AI
Jun 6, 2025
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.
[nitpick] Consider removing or rephrasing informal language and emoticons (e.g. ';)') in the contact page to maintain a more professional tone.
| Contact us about... pretty much Anything Azure related - or Rugby World Cup related ;) | |
| Contact us about anything related to Azure or the Rugby World Cup. |
Successfully rebuilt the ASP.NET Core MVC Slots application using Node.js Express framework in a new
/newfolder as requested.Changes Made
New Node.js Application Structure
/newfolderpackage.jsonand dependenciesTechnology Stack Migration
Features Preserved
/), About (/home/about), Contact (/home/contact), Privacy (/home/privacy)File Structure
Configuration
The Node.js app uses environment variables for configuration:
PORT- Server port (default: 3000)SITE- Site name (default: "AZ-2008")COLOUR- Background color (default: "#f8f9fa")DBCONN- Database connection stringAUTH- Authentication URLINSIGHTS- Application Insights connection stringTesting
Verified all functionality works correctly:
The new Node.js application provides identical functionality to the original ASP.NET Core version while running on the Express.js framework as requested.
Fixes #36.
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.