-
Notifications
You must be signed in to change notification settings - Fork 4k
Re-arrange OpenAPI Scopes for OAuth2 #3799
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
Conversation
🦋 Changeset detectedLatest commit: c87eda1 The changes in this PR will be included in the next version bump. This PR includes changesets to release 2 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
There was a problem hiding this 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 reorganizes how OAuth2 scopes are displayed in the OpenAPI documentation interface by introducing a new "available scopes" section that shows all scopes defined in OAuth2 flows when no specific scopes are required for a security scheme.
Key Changes:
- Added "available_scopes" translation key across all supported languages
- Modified
OpenAPISchemaScopesto support both required and available scopes display modes - Updated OAuth2 security display logic to show available scopes when no specific scopes are required
Reviewed Changes
Copilot reviewed 12 out of 12 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| packages/react-openapi/src/OpenAPIRequiredScopes.tsx | Made OpenAPISchemaScopes component public and added support for OAuth2 available scopes display mode |
| packages/react-openapi/src/OpenAPISecurities.tsx | Added logic to display available scopes from OAuth2 flows when no specific scopes are required |
| packages/react-openapi/src/translations/*.ts | Added "available_scopes" translation key for 8 languages (en, de, es, fr, ja, nl, no, pt-br, zh) |
| .changeset/calm-shirts-tie.md | Added changeset documenting the OAuth2 scopes reorganization |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
The latest updates on your projects. Learn more about Argos notifications ↗︎
|
Only show endpoint scopes in Required scopes. For OAuth2, add Available scopes back to the Authorization section if no endpoint scopes are present.