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

Split MapStaticAssets and UseStaticFiles into 2 docs #34080

Open
Rick-Anderson opened this issue Nov 12, 2024 · 6 comments
Open

Split MapStaticAssets and UseStaticFiles into 2 docs #34080

Rick-Anderson opened this issue Nov 12, 2024 · 6 comments
Assignees
Labels
aspnet-core/svc fundamentals/subsvc seQUESTered Identifies that an issue has been imported into Quest. Source - Docs.ms Docs Customer feedback via GitHub Issue ⌚ Not Triaged

Comments

@Rick-Anderson
Copy link
Contributor

Rick-Anderson commented Nov 12, 2024

Description

Per #34002, split them via:

  1. remove all the UseStaticFiles from this doc. set uid: fundamentals/mapStaticFiles Probably rename the file MapStaticAssets.md ( unless it messes up the PR files diff)
  2. Copy the V8 version ~/fundamentals/static-files.md into UseStaticFiles.md

The above will make for the most productive changes to review.
After PR merges, hook up the TOC for mapStaticFiles

Review UseStaticFiles history to see what UseStaticFiles changes were made from .NET 8 to .NET 9. I think only a few bug fixes.

@tdykstra @guardrex please review.

Page URL

https://learn.microsoft.com/en-us/aspnet/core/fundamentals/static-files?view=aspnetcore-9.0

Content source URL

https://github.com/dotnet/AspNetCore.Docs/blob/main/aspnetcore/fundamentals/static-files.md

Document ID

3fec6e08-fc99-7a5c-796f-3f2347cad891

Article author

@Rick-Anderson

Related Issues


Associated WorkItem - 343087

@Rick-Anderson Rick-Anderson added ⌚ Not Triaged reQUEST Triggers an issue to be imported into Quest Source - Docs.ms Docs Customer feedback via GitHub Issue labels Nov 12, 2024
@Rick-Anderson Rick-Anderson self-assigned this Nov 12, 2024
@sequestor sequestor bot added seQUESTered Identifies that an issue has been imported into Quest. and removed reQUEST Triggers an issue to be imported into Quest labels Nov 12, 2024
@guardrex
Copy link
Collaborator

The UID/file naming doesn't match the usual convention for lowercase, possibly with dashes. Should it be more like ...

  • MapStaticFiles
    • UID: fundamentals/mapstaticfiles or fundamentals/map-static-files
    • Filename: mapstaticfiles.md or map-static-files.md
  • UseStaticFiles
    • UID: fundamentals/usestaticfiles or fundamentals/use-static-files
    • Filename: usestaticfiles.md or use-static-files.md

@tdykstra
Copy link
Contributor

tdykstra commented Nov 12, 2024

The UID/file naming doesn't match the usual convention for lowercase, possibly with dashes. Should it be more like ...

  • MapStaticFiles

    • UID: fundamentals/mapstaticfiles or fundamentals/map-static-files
    • Filename: mapstaticfiles.md or map-static-files.md
  • UseStaticFiles

    • UID: fundamentals/usestaticfiles or fundamentals/use-static-files
    • Filename: usestaticfiles.md or use-static-files.md

I vote for map-static-files and use-static-files UID and file name. Whatever we do, I don't think we want upper case letters in UID or file name.

@tdykstra
Copy link
Contributor

Comments on the procedural text here for splitting the article into two:

  • It doesn't seem to take into account what Dan said about wanting to move much of the Blazor article text to the base article.
  • A lot of the text is devoted to comparing Map... vs. Use..., which will make splitting the text up into two well-organized articles more complicated than these instructions suggest.
  • It might make sense to divide up the remaining Blazor-specific content into Map and Use sections and work them into Blazor sections in the new Map and Use articles.

@guardrex
Copy link
Collaborator

guardrex commented Nov 13, 2024

move much of the Blazor article text to the base article

divide up the remaining Blazor-specific content into Map and Use sections and work them into Blazor sections in the new Map and Use articles.

Blazor article text that applies to all ASP.NET Core static file scenarios should move. Much of what's in the Blazor article is specific to Blazor, and it shouldn't be moved into the main doc set articles. I floated the idea that Blazor fundamentals content (all of it, not just for static files) should be moved entirely to the main doc set Fundamentals node (somehow), but no 🎲🎲 on that suggestion. I suppose it would be efficient if I made a set of HTML comments in that text for what can be moved and what has to stay. I've opened #34098, and I'll try to get that done today but certainly by EOW at the latest.

@Rick-Anderson
Copy link
Contributor Author

The UID/file naming doesn't match the usual convention for lowercase, possibly with dashes. Should it be more like ...

  • MapStaticFiles

    • UID: fundamentals/mapstaticfiles or fundamentals/map-static-files
    • Filename: mapstaticfiles.md or map-static-files.md
  • UseStaticFiles

    • UID: fundamentals/usestaticfiles or fundamentals/use-static-files
    • Filename: usestaticfiles.md or use-static-files.md

I vote for map-static-files and use-static-files UID and file name. Whatever we do, I don't think we want upper case letters in UID or file name.

For sure I'm not going to change the original UID which is a PK, no reason. There's no advantage to following a semi-convention on the implementation of the PK. I'll follow the convention for mapstaticfiles, UID: fundamentals/mapstaticfiles

Comments on the procedural text here for splitting the article into two:

  • It doesn't seem to take into account what Dan said about wanting to move much of the Blazor article text to the base article.

@guardrex will handle the Blazor changes

  • A lot of the text is devoted to comparing Map... vs. Use..., which will make splitting the text up into two well-organized articles more complicated than these instructions suggest.

No, it makes it easier, all that goes to the usestaticfiles file with a pointer to that in the mapstaticfiles files. mapstaticfiles handles the majority use case. If you need something that can only be done with usestaticfiles, that's in another doc.

  • It might make sense to divide up the remaining Blazor-specific content into Map and Use sections and work them into Blazor sections in the new Map and Use articles.

I'll let @guardrex refactor the Blazor-specific content

@guardrex
Copy link
Collaborator

I'll cut out 🔪 the offending bits from the Blazor article, but I'll wait until the split takes place here and new Map Static Files article goes into place. That way, I can cross-link to it.

It's also probably best for me to wait so that you can see what I have there. I understand you're going to two articles, while the single Blazor article has them both, but you can at least lift any of the text that you like before I get to cutting 🔪🔪 🔪 🔪 on the article.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
aspnet-core/svc fundamentals/subsvc seQUESTered Identifies that an issue has been imported into Quest. Source - Docs.ms Docs Customer feedback via GitHub Issue ⌚ Not Triaged
Projects
Development

No branches or pull requests

3 participants