kontent-ai / delivery-sdk-net

Kontent.ai Delivery .NET SDK
https://www.nuget.org/packages/Kontent.Ai.Delivery
MIT License
32 stars 43 forks source link

.NET 7 support #360

Closed novakv541 closed 1 year ago

novakv541 commented 1 year ago

May you migrate this project to .NET 7, please?

Thanks

Simply007 commented 1 year ago

Hello @novakv541,

thanks for the issue. May I ask you to add the information asked in the template? I know it seems boring, but the provided information would help us decide how to approach the issue.

The template is following:

### Motivation

Why is this feature required? What problems does it solve?

### Proposed solution

An ideal solution for the above problems.

### Additional context

Add any other context, screenshots, or reference links about the feature request here.
Simply007 commented 1 year ago

Linking related issues:

novakv541 commented 1 year ago

Motivation

.NET 6 is missing functionalities that .NET 7 has (caching) that we want to use for our .NET based Kotnent.ai project.

Proposed solution

Upgrade to .NET 7.

Additional context

https://www.honlsoft.com/blog/2022-09-24-output-caching-in-asp-net-core

novakv541 commented 1 year ago

@Simply007 May I ask about your prioritization, please? We need to start developing ASAP.

Thanks

Simply007 commented 1 year ago

Motivation

.NET 6 is missing functionalities that .NET 7 has (caching) that we want to use for our .NET based Kotnent.ai project.

Proposed solution

Upgrade to .NET 7.

Additional context

https://www.honlsoft.com/blog/2022-09-24-output-caching-in-asp-net-core

Correct me if I am wrong, but you can create .NET 7 application and use .NET 6 targetted libraries.

I have just tested out .NET 7 MVC application using Delivery SDK as well as registering 2 first routes from the Caching article. Everything is working just fine.

The compatibility should be OK by the official breaking change statement my MS.

novakv541 commented 1 year ago

Hi, We've tested it also and confirmed that it's working.

Thanks