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

new doc: Razor Pages Areas #5407

Closed
Rick-Anderson opened this issue Feb 9, 2018 · 8 comments
Closed

new doc: Razor Pages Areas #5407

Rick-Anderson opened this issue Feb 9, 2018 · 8 comments
Assignees
Labels
Help wanted Up for grabs. We would accept a PR to help resolve this issue

Comments

@Rick-Anderson
Copy link
Contributor

Rick-Anderson commented Feb 9, 2018

aspnet/Mvc#6926, aspnet/Mvc#6604
Scaffold the Identity UI and see how it uses areas.

Rename Areas in ASP.NET Core to Areas in ASP.NET Core for controllers
Before the intro, say this is for MVC controllers, see Areas for Razor Pages.

@Rick-Anderson Rick-Anderson added this to the 2018 - Quarter 1 milestone Feb 9, 2018
@Rick-Anderson Rick-Anderson self-assigned this Feb 9, 2018
@scottaddie scottaddie added the Help wanted Up for grabs. We would accept a PR to help resolve this issue label Mar 2, 2018
@Rick-Anderson Rick-Anderson removed their assignment Mar 14, 2018
@guardrex
Copy link
Collaborator

Described by @DamianEdwards on the Standup 👉 https://youtu.be/-EgE79M3tuc?t=34m00s

@Rick-Anderson
Copy link
Contributor Author

@guardrex @scottaddie who wants this?

@Rick-Anderson Rick-Anderson removed this from the Sprint 141 (9/3 to 9/21) milestone Aug 28, 2018
@mhmd-azeez
Copy link

@Rick-Anderson Can I take this? Is there anything I need to know first?

@Rick-Anderson
Copy link
Contributor Author

@Encrypt0r yes. Post an outline of the article here. @scottaddie do you have an idea for the outline?

@scottaddie
Copy link
Member

I haven't had time to look at this yet. @Encrypt0r Please provide your thoughts on a rough outline, and we'll review it.

@mhmd-azeez
Copy link

@scottaddie @Rick-Anderson
are you sure it's necessary to create a new page for "Razor Pages Areas"? Because all of the topics discussed in the current article must be duplicated in the new article as well. We can easily expand on the current page to include relevant information about "Razor pages" in the appropriate sections. This allows the users to see the whole picture more easily. Separating them might cause confusion.

Here are the changes that I think are needed:

Area features:

  1. Add information about Razor Pages wherever appropriate

A typical folder structure for that using MVC areas looks like below:

  1. Add "Pages" folder and some example pages to the folder structure

When MVC tries to render a view in an Area, by default, it tries to look in the following locations:

  1. Add paths for pages in the listed locations

Link Generation

  1. Add examples for Razor Pages.

  2. Edit the article to make sure the flow between the topics are smooth

@Rick-Anderson
Copy link
Contributor Author

Rick-Anderson commented Oct 4, 2018

@Encrypt0r you're right. It's much better to have one topic on areas and update it so it applies to RP. Like you say

Add information about Razor Pages wherever appropriate

You're approved! Nice outline of proposed changes. I think you've hit everything.

@Rick-Anderson
Copy link
Contributor Author

Moved to #9041

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Help wanted Up for grabs. We would accept a PR to help resolve this issue
Projects
None yet
Development

No branches or pull requests

4 participants