-
Notifications
You must be signed in to change notification settings - Fork 5
🚀[Feature]: Permission infrastructure + Track GitHub App Installation status #513
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
Merged
Merged
Changes from all commits
Commits
Show all changes
43 commits
Select commit
Hold shift + click to select a range
dacf107
Initial plan
Copilot 6564084
Add status tracking to GitHubAppInstallation class to compare permiss…
Copilot 439519d
Add Status property to GitHubAppInstallation display formats and upda…
Copilot 4bffffe
Fix app link
MariusStorhaug 896aaac
🩹 [Patch]: Enhance argument completers for GitHub configuration and u…
MariusStorhaug 5dfd0f5
🩹 [Patch]: Make the Context parameter mandatory in Switch-GitHubConte…
MariusStorhaug 6447faa
Add initial test scripts for GitHub API interactions
MariusStorhaug 6dc3cc6
🩹 [Patch]: Update Permissions property type to GitHubPermission array…
MariusStorhaug d3285a6
🩹 [Patch]: Add GitHubPermission.Format.ps1xml for GitHubPermission ta…
MariusStorhaug c402d1f
🩹 [Patch]: Enhance GitHubPermission initialization to handle unknown …
MariusStorhaug 1fc35cf
🩹 [Patch]: Refactor permission handling in GitHubApp and GitHubAppIns…
MariusStorhaug bfe5279
Fix formatting in NewFullCatalog function for better readability of s…
MariusStorhaug 2c60136
🩹 [Patch]: Simplify return statement in NewFullCatalog function for c…
MariusStorhaug 32710e4
🩹 [Patch]: Update permissions assignment in GitHubAppInstallationCont…
MariusStorhaug a8097a4
🩹 [Patch]: Refactor permission handling to use newPermissionList for …
MariusStorhaug 968dce5
🩹 [Patch]: Add new permission definition for enterprise custom organi…
MariusStorhaug 6ce56a6
🩹 [Patch]: Remove 'read' option from permission definition for improv…
MariusStorhaug 8d245b6
🩹 [Patch]: Remove CompareWithAppConfiguration method and replace with…
MariusStorhaug 4b901dd
🩹 [Patch]: Update permission filtering for 'User' scope to only inclu…
MariusStorhaug 4172bab
🩹 [Patch]: Update permission validation in tests to ensure correct ty…
MariusStorhaug 68c1bd7
🩹 [Patch]: Refactor permission list creation in GitHubPermission clas…
MariusStorhaug 4b67e3d
🩹 [Patch]: Remove unused variables related to permissions in GitHubPe…
MariusStorhaug db204ad
🩹 [Patch]: Add default case to permission filtering in GitHubPermissi…
MariusStorhaug 51eed1f
🩹 [Patch]: Enhance logging in Apps.Tests by adding permission context…
MariusStorhaug 54f4957
🩹 [Patch]: Refactor GitHub app tests for improved structure and clari…
MariusStorhaug fa601ec
🩹 [Patch]: Fix logging in installation tests to output individual ins…
MariusStorhaug ee7477c
🩹 [Patch]: Refactor permission checks in Apps.Tests to use a single v…
MariusStorhaug fb83868
🩹 [Patch]: Refactor permission list creation in GitHubPermission clas…
MariusStorhaug 2e46e9d
Refactor NewPermissionList methods to improve property handling from …
MariusStorhaug 53e5249
🩹 [Patch]: Refactor NewPermissionList method calls to use consistent …
MariusStorhaug 2a1944b
🩹 [Patch]: Simplify response handling in Get-GitHubAuthenticatedApp b…
MariusStorhaug 20f713f
🩹 [Patch]: Simplify GitHubPermission constructor and NewPermissionLis…
MariusStorhaug ac1a3a7
🩹 [Patch]: Update NewPermissionList methods to return GitHubPermissio…
MariusStorhaug 5f8a82e
Refactor NewPermissionList methods to streamline object property acce…
MariusStorhaug fe0dae8
🩹 [Patch]: Update Get-GitHubPermissionDefinition to use List property…
MariusStorhaug 3529d91
🩹 [Patch]: Enhance app context tests to verify Permissions property a…
MariusStorhaug b4cfbda
🩹 [Patch]: Remove skip condition from app context permission tests to…
MariusStorhaug f72dad0
🩹 [Patch]: Update app context tests to use Format-Table for permissio…
MariusStorhaug 1449987
🩹 [Patch]: Simplify permission validation by directly iterating over …
MariusStorhaug 058bdce
Add initial test scripts for GitHub API interactions
MariusStorhaug c5d8ad0
🩹 [Patch]: Clarify comment for Permissions property in GitHubAppInsta…
MariusStorhaug c4021cc
🩹 [Patch]: Replace ScriptBlock with PropertyName for Status column in…
MariusStorhaug 9041814
🩹 [Patch]: Update argument completers to use GitHubPermissionDefiniti…
MariusStorhaug File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.