Skip to content
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

Upgrade SA to 8LTS #732

Closed
wants to merge 15 commits into from
Closed

Upgrade SA to 8LTS #732

wants to merge 15 commits into from

Conversation

msalemcode
Copy link
Contributor

Description

This PR updates the SaaS Accelerator project to use .NET 8 LTS. This upgrade ensures that the project benefits from the latest features, performance improvements, and security updates provided by .NET 8 LTS.

Changes Made

  1. Project File Updates:

    • Updated the target framework in the project files to net8.0.
  2. Package Updates:

    • Updated NuGet packages to versions compatible with .NET 8 LTS.
  3. Code Adjustments:

    • Made necessary code changes to ensure compatibility with .NET 8 LTS.
    • Refactored deprecated APIs and replaced them with their .NET 8 LTS equivalents.
  4. Configuration Updates:

    • Updated configuration files, such as appsettings.json and appsettings.test.json, to align with .NET 8 LTS best practices.

Impact

  • Performance Improvements: Leveraging the performance enhancements in .NET 8 LTS.
  • Security Enhancements: Utilizing the latest security features and updates in .NET 8 LTS.
  • Long-Term Support: Ensuring the project is on a supported version of .NET for the foreseeable future.

Testing

  • Unit Tests: All existing unit tests have been run and updated where necessary to ensure compatibility with .NET 8 LTS.
  • Integration Tests: Conducted integration tests to verify that all components of the SaaS Accelerator work seamlessly with .NET 8 LTS.
  • Manual Testing: Performed manual testing of critical paths to ensure no regressions or issues were introduced.

Conclusion

This PR upgrades the SaaS Accelerator to .NET 8 LTS, bringing in the latest improvements and ensuring long-term support. The update includes necessary code adjustments, package updates, and configuration changes to maintain compatibility and leverage the benefits of .NET 8 LTS.

@msalemcode msalemcode linked an issue Jul 19, 2024 that may be closed by this pull request
@santhoshb-msft santhoshb-msft linked an issue Aug 13, 2024 that may be closed by this pull request
Copy link
Collaborator

Choose a reason for hiding this comment

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

line 505 and 517

az webapp create -g $ResourceGroupForDeployment -p $WebAppNameService -n $WebAppNameAdmin --runtime dotnet:6 --output $azCliOutput

update to dotnet:8

Copy link
Contributor

Choose a reason for hiding this comment

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

@msalemcode msalemcode closed this Aug 15, 2024
@msalemcode
Copy link
Contributor Author

Moving to PR #749

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.

.NET 8 Support Upgrade all projects to .NET 8
3 participants