Skip to content

Investigate the use of discriminated union in lieu of ActionResult<T> #27890

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

Closed
pranavkm opened this issue Nov 16, 2020 · 2 comments
Closed

Investigate the use of discriminated union in lieu of ActionResult<T> #27890

pranavkm opened this issue Nov 16, 2020 · 2 comments
Labels
area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates blocked The work on this issue is blocked due to some dependency feature-mvc-execution-pipeline Features related to how MVC executes a controller or razor page
Milestone

Comments

@pranavkm
Copy link
Contributor

The feature is still in proposal but if it lands, we should investigate replacing ActionResult<T> with a compiler feature.

Current spec: https://github.com/dotnet/csharplang/blob/master/proposals/discriminated-unions.md

@pranavkm pranavkm added area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates blocked The work on this issue is blocked due to some dependency labels Nov 16, 2020
@pranavkm pranavkm added this to the 6.0.0 milestone Nov 16, 2020
@javiercn javiercn added the feature-mvc-execution-pipeline Features related to how MVC executes a controller or razor page label Apr 18, 2021
@ghost
Copy link

ghost commented Apr 21, 2021

Thanks for contacting us.
We're moving this issue to the Next sprint planning milestone for future evaluation / consideration. Because it's not immediately obvious that this is a bug in our framework, we would like to keep this around to collect more feedback, which can later help us determine the impact of it. We will re-evaluate this issue, during our next planning meeting(s).
If we later determine, that the issue has no community involvement, or it's very rare and low-impact issue, we will close it - so that the team can focus on more important and high impact issues.
To learn more about what to expect next and how this issue will be handled you can read more about our triage process here.

@pranavkm
Copy link
Contributor Author

Closing this. The feature isn't planned for C# 10. We can file a new issue at a later date.

@ghost ghost locked as resolved and limited conversation to collaborators May 21, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates blocked The work on this issue is blocked due to some dependency feature-mvc-execution-pipeline Features related to how MVC executes a controller or razor page
Projects
None yet
Development

No branches or pull requests

3 participants