Skip to content

Remove ApiAuthorization.IdentityServer #49193

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Jul 10, 2023
Merged

Remove ApiAuthorization.IdentityServer #49193

merged 3 commits into from
Jul 10, 2023

Conversation

halter73
Copy link
Member

@halter73 halter73 commented Jul 4, 2023 •
edited
Loading

Remove Microsoft.AspNetCore.ApiAuthorization.IdentityServer

#46131

@halter73 halter73 requested a review from javiercn July 4, 2023 01:19
@ghost ghost added the area-identity Includes: Identity and providers label Jul 4, 2023
@halter73 halter73 force-pushed the halter73/46131 branch 4 times, most recently from 094bea1 to b331911 Compare July 6, 2023 03:57
@halter73 halter73 marked this pull request as ready for review July 6, 2023 03:59
@halter73 halter73 requested review from a team and wtgodbe as code owners July 6, 2023 03:59
@halter73 halter73 enabled auto-merge (squash) July 6, 2023 03:59
wtgodbe
wtgodbe reviewed Jul 7, 2023
View reviewed changes
wtgodbe
wtgodbe reviewed Jul 7, 2023
View reviewed changes
wtgodbe
wtgodbe reviewed Jul 7, 2023
View reviewed changes
wtgodbe
wtgodbe requested changes Jul 7, 2023
View reviewed changes
Copy link
Member

@wtgodbe wtgodbe left a comment

Choose a reason for hiding this comment

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

Couple small changes + undo the submodule change

@halter73
Copy link
Member Author

@wtgodbe Is this good to merge?

@wtgodbe wtgodbe disabled auto-merge July 10, 2023 17:16
@ghost ghost added the area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework label Jul 10, 2023
@wtgodbe wtgodbe merged commit a5a0ede into main Jul 10, 2023
@wtgodbe wtgodbe deleted the halter73/46131 branch July 10, 2023 17:17
@ghost ghost added this to the 8.0-preview7 milestone Jul 10, 2023
@halter73 halter73 mentioned this pull request Mar 20, 2024
Merged
@martincostello martincostello mentioned this pull request May 1, 2024
Closed
1 task
@omarceloribeiro omarceloribeiro mentioned this pull request Nov 13, 2024
Open
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wtgodbe wtgodbe wtgodbe approved these changes

@javiercn javiercn Awaiting requested review from javiercn

Assignees
No one assigned
Labels
area-identity Includes: Identity and providers area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework
Projects
None yet
Milestone
8.0-preview7
Development

Successfully merging this pull request may close these issues.
2 participants
@halter73 @wtgodbe