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

Nickdrouin/net80 #20

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Conversation

NickDrouin
Copy link

I created a new project using the latest net8.0 tooling for server-side Blazor. As such, many of the pages moved around within the repo, with no significant functional changes.

Note: The Weather controller now forces Authorization, I felt this wasn't demonstrated in the original sample and is a common use-case for devs.

The Program.cs both has changes to match the latest dotnet new template, but also has a work-around for:
okta/okta-aspnet#260
This commit should be dropped once the issue is fixed in the okta-aspnet lib. Meanwhile, it can be considered by other devs as a work-around.

@davidcorbin-atmosera
Copy link

@NickDrouin - with 4.6.1 now released, does it make sense to update this?

@NickDrouin
Copy link
Author

@NickDrouin - with 4.6.1 now released, does it make sense to update this?

Thanks @davidcorbin-atmosera , I updated the lib and removed the work-around.

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