dotnet / dotnet/aspnetcore

Consider making `IdentityResult.Failed(List<IdentityError>? errors)` public

Open
#53,693 1 comment 0 reactions 0 assignees View on GitHub
area-identity enhancement
Dominant language
C#
Stars
38.4k
Forks
10.9k
Avg merge
2d 10h
Merged PRs (30d)
281

Description

### Is there an existing issue for this?

- [x] I have searched the existing issues

### Is your feature request related to a problem? Please describe the problem.

I wasn't sure if this should be labeled as API Change or Design Proposal. `ItentityResult.Failed(params IdentityError[] errors)` is public, but if you have built a `List`, then using that method means converting your list to array. There is `IdentityResult.Failed(List? errors)`, but for some reason it's marked internal, and looking at the source code, I don't see a reason it couldn't be public.

### Describe the solution you'd like

Change `IdentityResult.Failed(List? errors)` from `internal` to `public`.

### Additional context

_No response_

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.