Skip to content

Network - 25384 - Application admin rights are constrained to specific Private Access apps, not tenant-wide #722

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.

Sign up for GitHub

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

Jump to bottom
Open
KManoj0000 wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Network - 25384 - Application admin rights are constrained to specific Private Access apps, not tenant-wide #722

KManoj0000 wants to merge 4 commits into from
+369 −0

Conversation

@KManoj0000
Copy link
Collaborator

@KManoj0000 KManoj0000 commented Dec 23, 2025

Application admin rights are constrained to specific Private Access apps, not tenant-wide

@KManoj0000 KManoj0000 self-assigned this Dec 23, 2025
@KManoj0000 KManoj0000 changed the title Feature-25384 Network - 25384 - Application admin rights are constrained to specific Private Access apps, not tenant-wide Dec 23, 2025
@KManoj0000 KManoj0000 added enhancement New feature or request ready for review PR is ready for review and merging labels Dec 23, 2025
@KManoj0000 KManoj0000 marked this pull request as draft December 24, 2025 09:52
@KManoj0000 KManoj0000 closed this Dec 24, 2025
@KManoj0000 KManoj0000 reopened this Dec 30, 2025
@KManoj0000 KManoj0000 marked this pull request as ready for review December 30, 2025 02:22
alexandair
alexandair requested changes Dec 30, 2025
View reviewed changes
Copy link
Collaborator

@alexandair alexandair left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@KManoj0000 Please, address my feedback.

@KManoj0000
Copy link
Collaborator Author

KManoj0000 commented Jan 2, 2026 •
edited
Loading

@KManoj0000 Please, address my feedback.

@alexandair I've addressed the feedback, please re-review again

@alexandair alexandair self-requested a review January 2, 2026 11:01
alexandair
alexandair requested changes Jan 2, 2026
View reviewed changes
Copy link
Collaborator

@alexandair alexandair left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@KManoj0000 Please, address my feedback.

@KManoj0000
Copy link
Collaborator Author

KManoj0000 commented Jan 2, 2026

@KManoj0000 Please, address my feedback.

@alexandair I've addressed the feedback, and made requested changes please check and review

@KManoj0000 KManoj0000 requested a review from alexandair January 2, 2026 11:59
@alexandair
Copy link
Collaborator

alexandair commented Jan 2, 2026

@KManoj0000 Have you tested it after the suggested changes? Do we have updated screenshots (hopefully no need if they are the same)?

@KManoj0000
Copy link
Collaborator Author

KManoj0000 commented Jan 2, 2026

@KManoj0000 Have you tested it after the suggested changes? Do we have updated screenshots (hopefully no need if they are the same)?

@alexandair Yes I've tested with the changes and after the changes, code execution speed has been increased, but the output is same.

@alexandair
Copy link
Collaborator

alexandair commented Jan 2, 2026

@KManoj0000 Have you tested it after the suggested changes? Do we have updated screenshots (hopefully no need if they are the same)?

@alexandair Yes I've tested with the changes and after the changes, code execution speed has been increased, but the output is same.

Excellent.

alexandair
alexandair approved these changes Jan 2, 2026
View reviewed changes
Copy link
Collaborator

@alexandair alexandair left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alexandair alexandair alexandair approved these changes

@merill merill Awaiting requested review from merill

Assignees

@KManoj0000 KManoj0000

Labels

enhancement New feature or request ready for review PR is ready for review and merging

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@KManoj0000 @alexandair