Skip to content

Conversation

@gao-sun
Copy link
Member

@gao-sun gao-sun commented Dec 10, 2025

Summary

add "Handle user permission change" section content, resolves LOG-11792

Copy link
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

This PR adds comprehensive documentation for handling user permission changes in authorization flows by creating a reusable MDX fragment and integrating it across multiple authorization documentation pages.

  • Creates a new reusable documentation fragment explaining how permission changes affect JWT tokens in both global and organization contexts
  • Integrates the fragment into three authorization documentation files, replacing a "work in progress" placeholder in one location
  • Adds @swc/core to the onlyBuiltDependencies configuration in package.json

Reviewed changes

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

Show a summary per file
File Description
package.json Adds @swc/core to onlyBuiltDependencies to ensure it's built during installation
docs/authorization/fragments/_handle-user-permission-change.mdx New reusable fragment documenting permission change handling with type-specific content for global vs organization contexts
docs/authorization/organization-permissions.mdx Integrates the new permission change fragment for organization permissions
docs/authorization/organization-level-api-resources.mdx Integrates the new permission change fragment for organization-level API resources
docs/authorization/global-api-resources.mdx Replaces "work in progress" placeholder with the new permission change fragment for global API resources

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

@gao-sun gao-sun merged commit 96bf4ce into master Dec 10, 2025
23 of 24 checks passed
@gao-sun gao-sun deleted the gao-update-permission-change-section branch December 10, 2025 08:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants