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

RP conventions features update #5520

Closed
guardrex opened this issue Feb 20, 2018 · 5 comments
Closed

RP conventions features update #5520

guardrex opened this issue Feb 20, 2018 · 5 comments
Assignees

Comments

@guardrex
Copy link
Collaborator

guardrex commented Feb 20, 2018

mvc/razor-pages/razor-pages-convention-features.md
https://docs.microsoft.com/en-us/aspnet/core/mvc/razor-pages/razor-pages-convention-features

IPageHandlerModelConvention

@guardrex guardrex added the 2.1 label Feb 20, 2018
@guardrex guardrex added this to the 2018 - Quarter 1 milestone Feb 20, 2018
@guardrex guardrex self-assigned this Feb 20, 2018
@guardrex guardrex changed the title RP conventions features: RP conventions features update Feb 20, 2018
@guardrex
Copy link
Collaborator Author

guardrex commented Apr 12, 2018

@guardrex
Copy link
Collaborator Author

@pranavkm Looks like this should be focused on IPageHandlerModelConvention ... not IPagePropertyModelConvention.

@pranavkm
Copy link
Contributor

@guardrex might be good to add a quip about property model \ parameter model conventions. Sort of along the lines of https://docs.microsoft.com/en-us/aspnet/core/mvc/controllers/application-model?view=aspnetcore-2.1#sample-modifying-the-parametermodel. For most parts, you could copy the docs \ samples verbatim.

@guardrex
Copy link
Collaborator Author

@pranavkm Where we stand today with this topic is that it focuses on conventions that implement IPageConvention ... so it's very RP options.Conventions-feature focused.

Given this issue is only to cover new 2.1 IPageConvention-based features (and there seems to be only one), let's address that ☝️ separately. #5958

@pranavkm
Copy link
Contributor

Okies, sounds good.

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

No branches or pull requests

2 participants