Skip to content

Fix description of the apply argument on the @authorize directive#9404

Merged
glen-84 merged 1 commit intomainfrom
gai/fix-apply-description
Mar 20, 2026
Merged

Fix description of the apply argument on the @authorize directive#9404
glen-84 merged 1 commit intomainfrom
gai/fix-apply-description

Conversation

@glen-84
Copy link
Copy Markdown
Member

@glen-84 glen-84 commented Mar 20, 2026

Summary of the changes (Less than 80 chars)

  • Fix description of the apply argument on the @authorize directive.

Copilot AI review requested due to automatic review settings March 20, 2026 16:14
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

Updates the documentation text for the apply argument on the @authorize GraphQL directive to correct a typo and accurately describe the default behavior.

Changes:

  • Fixes duplicated word “when when” in the apply argument description.
  • Updates the default-behavior text from “validation phase” to “before the resolver is executed”.
  • Regenerates affected GraphQL schema snapshot outputs to reflect the updated description.

Reviewed changes

Copilot reviewed 15 out of 15 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/HotChocolate/Core/src/Authorization/AuthorizeDirectiveType.cs Updates the apply argument description text on @authorize.
src/HotChocolate/Core/test/Authorization.Tests/snapshots/SchemaFirstAuthorizationTests.Authorize_Apply_Can_Be_Omitted.graphql Updates snapshot to match the corrected directive argument description.
src/HotChocolate/Core/test/Authorization.Tests/snapshots/AnnotationBasedAuthorizationTests.Authorize_Node_Field_Schema.graphql Updates snapshot to match the corrected directive argument description.
src/HotChocolate/AspNetCore/test/AspNetCore.Authorization.Tests/snapshots/AuthorizeDirectiveTests.TypeAuth_WithRoles.graphql Updates snapshot to match the corrected directive argument description.
src/HotChocolate/AspNetCore/test/AspNetCore.Authorization.Tests/snapshots/AuthorizeDirectiveTests.TypeAuth_WithPolicy.graphql Updates snapshot to match the corrected directive argument description.
src/HotChocolate/AspNetCore/test/AspNetCore.Authorization.Tests/snapshots/AuthorizeDirectiveTests.TypeAuth_DefaultPolicy.graphql Updates snapshot to match the corrected directive argument description.
src/HotChocolate/AspNetCore/test/AspNetCore.Authorization.Tests/snapshots/AuthorizeDirectiveTests.FieldAuth_WithRoles.graphql Updates snapshot to match the corrected directive argument description.
src/HotChocolate/AspNetCore/test/AspNetCore.Authorization.Tests/snapshots/AuthorizeDirectiveTests.FieldAuth_WithPolicy_Validation.graphql Updates snapshot to match the corrected directive argument description.
src/HotChocolate/AspNetCore/test/AspNetCore.Authorization.Tests/snapshots/AuthorizeDirectiveTests.FieldAuth_WithPolicy_BeforeResolver.graphql Updates snapshot to match the corrected directive argument description.
src/HotChocolate/AspNetCore/test/AspNetCore.Authorization.Tests/snapshots/AuthorizeDirectiveTests.FieldAuth_WithPolicy_AfterResolver.graphql Updates snapshot to match the corrected directive argument description.
src/HotChocolate/AspNetCore/test/AspNetCore.Authorization.Tests/snapshots/AuthorizeDirectiveTests.FieldAuth_WithPolicy.graphql Updates snapshot to match the corrected directive argument description.
src/HotChocolate/AspNetCore/test/AspNetCore.Authorization.Tests/snapshots/AuthorizeDirectiveTests.FieldAuth_DefaultPolicy_Validation.graphql Updates snapshot to match the corrected directive argument description.
src/HotChocolate/AspNetCore/test/AspNetCore.Authorization.Tests/snapshots/AuthorizeDirectiveTests.FieldAuth_DefaultPolicy_BeforeResolver.graphql Updates snapshot to match the corrected directive argument description.
src/HotChocolate/AspNetCore/test/AspNetCore.Authorization.Tests/snapshots/AuthorizeDirectiveTests.FieldAuth_DefaultPolicy_AfterResolver.graphql Updates snapshot to match the corrected directive argument description.
src/HotChocolate/AspNetCore/test/AspNetCore.Authorization.Tests/snapshots/AuthorizeDirectiveTests.FieldAuth_DefaultPolicy.graphql Updates snapshot to match the corrected directive argument description.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@glen-84 glen-84 merged commit 67b628e into main Mar 20, 2026
127 of 128 checks passed
@glen-84 glen-84 deleted the gai/fix-apply-description branch March 20, 2026 16:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants