Skip to content

[APIView] Endpoint definition to get language approvers and admins#15523

Open
AlitzelMendez wants to merge 2 commits intoAzure:mainfrom
AlitzelMendez:mcp-endpoint-admins-approvers
Open

[APIView] Endpoint definition to get language approvers and admins#15523
AlitzelMendez wants to merge 2 commits intoAzure:mainfrom
AlitzelMendez:mcp-endpoint-admins-approvers

Conversation

@AlitzelMendez
Copy link
Copy Markdown
Member

Endpoint definition for: #15414

Copilot AI review requested due to automatic review settings May 6, 2026 18:19
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

This PR adds TypeSpec/OpenAPI documentation for new “permissions” endpoints in APIView (language approvers + admin list) and wires the new spec into the TypeSpec entrypoint so it’s emitted with the rest of the external API surface.

Changes:

  • Added permissions.tsp defining /api/permissions/approvers/{language} and /api/permissions/admins endpoints under Bearer auth.
  • Added supporting models for the permissions endpoints (approver/admin info).
  • Imported the new TypeSpec file from main.tsp so it’s included in the generated OpenAPI.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
src/dotnet/APIView/APIViewWeb/typespec/permissions.tsp Adds TypeSpec definitions/models for permissions endpoints (approvers/admins) under /api/permissions.
src/dotnet/APIView/APIViewWeb/typespec/main.tsp Imports permissions.tsp so the endpoints are included in the compiled OpenAPI output.

Comment thread src/dotnet/APIView/APIViewWeb/typespec/permissions.tsp
Comment thread src/dotnet/APIView/APIViewWeb/typespec/permissions.tsp
Comment thread src/dotnet/APIView/APIViewWeb/typespec/permissions.tsp Outdated
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.

2 participants