-
Notifications
You must be signed in to change notification settings - Fork 10.4k
[release/8.0] Add ForgotPasswordRequest to STJ source generation context #52830
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
Conversation
Thanks for your PR, @Kahbazi. Someone from the team will get assigned to your PR shortly and we'll get it reviewed. |
Hi @Kahbazi. If this is not a tell-mode PR, please make sure to follow the instructions laid out in the servicing process document. |
Hi @Kahbazi. Please make sure you've updated the PR description to use the Shiproom Template. Also, make sure this PR is not marked as a draft and is ready-to-merge. To learn more about how to prepare a servicing PR click here. |
I've removed the |
Oops, evidently I jumped the gun. This has already been approved over email, but I'll certainly defer to @halter73's guidance. |
@mkArtakMSFT We should take it. It's a trivial bugfix which has already been approved by tactics. Thanks @amcasey |
Looks like this PR hasn't been active for some time and the codebase could have been changed in the meantime. |
@dotnet/aspnet-build Can someone merge this? @amcasey should be able to confirm this was approved December 14th. The email subject was "[release/8.0] Fix aspnetcore AOT bug". |
/azp run |
Azure Pipelines successfully started running 3 pipeline(s). |
Confirmed - tactics approved over email 12/14. |
Manual backport of #52714
Add ForgotPasswordRequest to STJ source generation context
Add
ForgotPasswordRequest
toIdentityEndpointsJsonSerializerContext
forApiEndpoints
.Description
Fixes #52646
Customer Impact
Regression?
Risk
Verification
Packaging changes reviewed?
/cc @halter73 @amcasey