Skip to content

Conversation

@devin-ai-integration
Copy link

@devin-ai-integration devin-ai-integration bot commented Sep 19, 2025

NET-15: Create .NET Core 6 Web Application Project in COG-GTM/SampleMvcWebApp

Summary

This PR implements the foundation phase of migrating the existing .NET Framework 4.5.1 MVC application to .NET 6. Creates a new SampleWebApp.Core project using modern .NET 6 patterns while preserving the existing application for parallel development and gradual migration.

Key Changes:

  • New .NET 6 Project: Created SampleWebApp.Core/ with SDK-style project format targeting net6.0
  • Modern Architecture: Implemented minimal hosting model with standard ASP.NET Core middleware pipeline
  • Configuration Migration: Migrated from Web.config to appsettings.json pattern with environment-specific configurations
  • Project Structure: Established standard .NET 6 folder conventions (Controllers/, Views/, Models/, wwwroot/)
  • Solution Integration: Added new project to existing solution with all build configurations
  • Documentation: Updated README with comprehensive migration progress tracking

The new project is intentionally minimal with basic HomeController returning simple content. Project references to DataLayer/ServiceLayer are commented out since those remain .NET Framework projects for now.

Review & Testing Checklist for Human

This is foundational infrastructure work with medium risk - please verify the following 2 items:

  • Build & Runtime Verification: Confirm SampleWebApp.Core builds successfully (dotnet build SampleWebApp.Core/SampleWebApp.Core.csproj) and runs in Development environment (dotnet run --project SampleWebApp.Core --environment Development). Test that http://localhost:[port] returns "SampleWebApp.Core - .NET 6 Web Application is running!"

  • Configuration Migration Accuracy: Compare SampleWebApp.Core/appsettings.json connection string and ApplicationSettings against original SampleWebApp/Web.config (lines 13-22) to ensure values were migrated correctly, especially the database connection string and HostTypeString/DatabaseLoginPrefix settings

Notes

  • Project references to DataLayer/ServiceLayer are intentionally commented out since those projects remain .NET Framework 4.5.1 and will be migrated in future phases
  • The existing .NET Framework application remains untouched and functional
  • Environment-specific configurations (Development/Staging/Production) use different logging levels as appropriate
  • This establishes the foundation for incremental migration of controllers, views, and business logic in subsequent phases

Session Details:

- Create new SampleWebApp.Core project with SDK-style format targeting net6.0
- Enable nullable reference types and implicit usings
- Implement minimal hosting model in Program.cs
- Set up standard .NET 6 folder structure (Controllers/, Views/, Models/, wwwroot/)
- Configure environment-specific appsettings files (Development, Staging, Production)
- Create Properties/launchSettings.json with multiple environment profiles
- Add project to existing solution file
- Project references to DataLayer/ServiceLayer commented out for future migration phase
- Project builds successfully with dotnet build

Co-Authored-By: Shawn Azman <shawn.d.azman@gmail.com>
@devin-ai-integration
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

@ShawnAzman
Copy link

Update the README with the progress

- Add comprehensive migration progress section at top of README
- Document completed Phase 1 foundation work (SampleWebApp.Core project)
- List next steps for ongoing migration phases
- Include project structure overview showing both old and new projects
- Reference NET-15 ticket and PR #1 for traceability
- Maintain original application documentation below migration section

Addresses GitHub PR comment requesting README update with progress.

Co-Authored-By: Shawn Azman <shawn.d.azman@gmail.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.

2 participants