Skip to content

Conversation

devin-ai-integration[bot]
Copy link

Migrate from ASP.NET MVC 5 to ASP.NET Core 7 Web API

Summary

This PR migrates the backend from ASP.NET MVC 5 (.NET Framework 4.8) to ASP.NET Core Web API while keeping the AngularJS frontend unchanged. Key changes include:

  • Created new SDK-style project file AspNetCore.WebApi.csproj targeting .NET 8.0
  • Replaced Web.config with appsettings.json for configuration
  • Replaced MVC controllers with API controllers returning JSON data
  • Implemented static file serving middleware for frontend assets
  • Converted Razor view to static HTML file
  • Added CORS support for frontend integration
  • Created detailed migration documentation in README-MIGRATION.md

The application was tested and confirmed working with all AngularJS components rendering properly.

Frontend Screenshot
API Endpoint
Landing API Endpoint

Review & Testing Checklist for Human

  • Verify that all AngularJS components load and function correctly in the browser
  • Check browser console for any JavaScript errors or 404s for static resources
  • Test API endpoints (/api/api/version, /api/api/health, /api/landingapi/config) for correct response format
  • Confirm that the target framework (.NET 8.0) meets project requirements (PR title mentions ASP.NET Core 7)
  • Review static file serving configuration for potential path issues across environments

Test Plan

  1. Build and run the application: dotnet run --project AspNetCore.WebApi.csproj
  2. Navigate to the application in browser
  3. Verify frontend components render properly showing jQuery and AngularJS version info
  4. Test API endpoints to confirm they return expected JSON data

Notes

  • The original requirement specified ASP.NET Core 7, but the implementation uses .NET 8.0 which is newer
  • No database migrations were needed as the application doesn't use a database
  • Static file paths maintain the same structure as the original MVC application

Requested by: kevin.tang@windsurf.com (@kevintang-windsurf)
Session link: https://app.devin.ai/sessions/d731023a40ad41808eb4c5f333c2564e

Co-Authored-By: kevin.tang@windsurf.com <kevin.tang@windsurf.com>
Copy link
Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@kevintang-windsurf
Copy link

Create a diagram showing the changes made

Co-Authored-By: kevin.tang@windsurf.com <kevin.tang@windsurf.com>
@kevintang-windsurf
Copy link

I want it to be a mermaid diagram and rendered properly

Co-Authored-By: kevin.tang@windsurf.com <kevin.tang@windsurf.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant