Skip to content

Add Api Permissions Tab - #12

Merged
StefH merged 15 commits into
mainfrom
del-api-permi
Sep 12, 2025
Merged

StefH merged 15 commits into
mainfrom
del-api-permi

Conversation

@StefH

@StefH StefH commented Sep 11, 2025

Copy link
Copy Markdown
Member

No description provided.

@StefH StefH self-assigned this Sep 11, 2025
@StefH
StefH requested a review from Copilot September 11, 2025 11:40
@StefH StefH added the enhancement New feature or request label Sep 11, 2025

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 an API Permissions tab to the Azure App Registrations Manager, allowing users to view and manage API permissions for app registrations. The implementation includes functionality for requesting resource access and managing OAuth2 permission grants.

Key changes:

  • Added new API Permissions tab with UI for viewing, requesting, and managing permissions
  • Refactored data loading to fetch permissions alongside application data for improved performance
  • Updated models to include permission-related properties and enterprise application references

Reviewed Changes

Copilot reviewed 25 out of 25 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
RequestResourceAccessHandler.cs New service for handling interactive OAuth2 token requests
ApiPermissions/*.cs New UI components and models for the API Permissions tab
AzureCommandsHandler.cs Major refactoring with new permission-related methods and parallel data loading
AppRegInfo.cs Added properties for enterprise applications and API permissions
MainWindow.xaml/cs Integration of new API Permissions tab and parallel data loading
Various other files Supporting changes for new functionality

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread AzureAppRegistrationsManager.WinUI/Services/AzureCommandsHandler.cs Outdated
Comment thread AzureAppRegistrationsManager.WinUI/Extensions/StringExtensions.cs
StefH and others added 2 commits September 12, 2025 09:57
…er.cs

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@StefH
StefH merged commit bb36237 into main Sep 12, 2025
1 check passed
@StefH
StefH deleted the del-api-permi branch September 12, 2025 08:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants