Skip to content

Move and overhaul the RP authz conventions article#36919

Merged
guardrex merged 4 commits into
mainfrom
guardrex/move-rp-authz-article
Mar 25, 2026
Merged

Move and overhaul the RP authz conventions article#36919
guardrex merged 4 commits into
mainfrom
guardrex/move-rp-authz-article

Conversation

@guardrex
Copy link
Copy Markdown
Collaborator

@guardrex guardrex commented Mar 25, 2026

Fixes #36916

Wade .....

  • Moved the samples and added a 10.0 sample for this.
  • Simplified the coverage and refactored the versioning ... NO more broken bookmark links in this one now. 🎉
  • Updated code snippet links to use modern triple-dot syntax.
  • Location change placed the article in the new, dedicated razor-pages/security/authorization node. Surfacing the article there with an additional resource link at the bottom of the Security > Authorization > Introduction article.
  • Made a redirection file entry for the new article location/filename.
  • Updated the ToC for the location change.
  • Also updating the UID to match (which IMO should've always been done ... and Scott Addie agreed at the time ... but we lost that fight 😢).
  • AI didn't make any suggestions on this one, so I'm not adding the AI-assist metadata. It's all 🦖 on this one! 😆

NOTE TO SELF 🦖

💀 the main doc set repo sample apps after review/approval. I'm not going to do it until then because it hoses the diff and makes the PR much harder to review.


Internal previews

📄 File 🔗 Preview link
aspnetcore/blazor/security/index.md aspnetcore/blazor/security/index
aspnetcore/razor-pages/index.md aspnetcore/razor-pages/index
aspnetcore/razor-pages/razor-pages-conventions.md aspnetcore/razor-pages/razor-pages-conventions
aspnetcore/razor-pages/security/authorization/conventions.md aspnetcore/razor-pages/security/authorization/conventions
aspnetcore/security/authorization/introduction.md aspnetcore/security/authorization/introduction
aspnetcore/security/authorization/policies.md aspnetcore/security/authorization/policies
aspnetcore/test/integration-tests.md aspnetcore/test/integration-tests
aspnetcore/toc.yml aspnetcore/toc

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Moves the Razor Pages authorization conventions content into the dedicated Razor Pages security/authorization section, updates inbound references across the docset, and adds a redirect for the old article location.

Changes:

  • Adds a new article at razor-pages/security/authorization/conventions and removes the old security/authorization/razor-pages-authorization article.
  • Updates TOC entries and cross-references throughout related Razor Pages, security/authorization, Blazor security, and testing docs.
  • Adds an OpenPublishing redirect from the deleted article path to the new URL.

Reviewed changes

Copilot reviewed 17 out of 17 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
aspnetcore/toc.yml Adds the new Razor Pages authorization conventions node and removes the old entry from Security > Authorization.
aspnetcore/test/integration-tests.md Updates the in-article link target to the new conventions article UID/anchor.
aspnetcore/test/integration-tests/includes/integration-tests5.md Updates the link target to the new conventions article UID/anchor.
aspnetcore/test/integration-tests/includes/integration-tests7.md Updates the link target to the new conventions article UID/anchor.
aspnetcore/test/integration-tests/includes/integration-tests8.md Updates the link target to the new conventions article UID/anchor.
aspnetcore/test/integration-tests/includes/integration-tests9.md Updates the link target to the new conventions article UID/anchor.
aspnetcore/security/authorization/policies.md Updates xref to point to the new Razor Pages conventions article.
aspnetcore/security/authorization/policies/includes/policies5.md Updates xref to point to the new Razor Pages conventions article.
aspnetcore/razor-pages/security/authorization/conventions.md New/overhauled article content in the new Razor Pages security/authorization location.
aspnetcore/razor-pages/razor-pages-conventions.md Updates “Additional resources” xrefs to the new conventions article.
aspnetcore/razor-pages/index.md Updates references/xrefs to the new conventions article.
aspnetcore/razor-pages/index/includes/index35.md Updates references/xrefs to the new conventions article.
aspnetcore/blazor/security/index.md Updates xref to the new conventions article (for Blazor + Razor Pages integration note).
aspnetcore/security/authorization/razor-pages-authorization.md Deletes the old article at the previous location/UID.
aspnetcore/security/authorization/razor-pages-authorization/samples/2.x/AuthorizationSample/README.md Updates the Learn link to the new article URL.
aspnetcore/security/authorization/razor-pages-authorization/samples/3.x/AuthorizationSample/README.md Updates the Learn link to the new article URL.
.openpublishing.redirection.json Adds a redirect from the deleted article path to the new conventions URL.

Comment thread aspnetcore/razor-pages/security/authorization/conventions.md
@guardrex guardrex requested a review from wadepickett March 25, 2026 17:03
Comment thread aspnetcore/razor-pages/security/authorization/conventions.md Outdated
Comment thread aspnetcore/razor-pages/security/authorization/conventions.md Outdated
Copy link
Copy Markdown
Contributor

@wadepickett wadepickett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved. Looks great! I found two very minor items for you to consider.

guardrex and others added 2 commits March 25, 2026 13:55
Co-authored-by: Wade Pickett <wpickett@microsoft.com>
@guardrex guardrex merged commit 315994b into main Mar 25, 2026
4 checks passed
@guardrex guardrex deleted the guardrex/move-rp-authz-article branch March 25, 2026 18:25
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

Successfully merging this pull request may close these issues.

Move article on Razor Pages authorization conventions

3 participants