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

Update .NET-version #116

Closed
wants to merge 1 commit into from
Closed

Update .NET-version #116

wants to merge 1 commit into from

Conversation

kleivane
Copy link
Contributor

@kleivane kleivane commented Sep 16, 2024

Update notnet version in readme, dotnet 8 is required is it not?

.NET 6.0 has end of support November 12, 2024

Dotnet 8 is required is it not?
@martinothamar
Copy link
Contributor

Yeah we should, there is an issue on it here: #2
It will involve updating projects and dependencies, base docker image etc..

@kleivane
Copy link
Contributor Author

I might be misunderstanding but, this readme is for running Altinn apps locally? We followed the readme yesterday with a new developer without a prior who installed .NET 6 as pr the docs and it failed demanding an upgrade to 8. After installing .NET8, it all went smooth. So I just thought the readme needed an update for new developers?

The issue #2 is about updating the dockerimage for localtest?

@martinothamar
Copy link
Contributor

Oh, I see, yeah so running localtest (the code in this repository) doesn't even require .NET as it runs inside a container runtime. The .NET/Altinn version of your app dictates which version of .NET should be used. v7 of the backend targets .NET 6 while v8 targets .NET 8. So I guess we could make the doc here more precise

@martinothamar
Copy link
Contributor

So essentially this one is a bit more descriptive: #109

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