Skip to content

[Permissions] Add Where-Used and Permissions Overview navigation actions#8947

Merged
Groenbech96 merged 3 commits into
mainfrom
user/magnushar/626501-permissions-overview-v2
Jul 20, 2026
Merged

[Permissions] Add Where-Used and Permissions Overview navigation actions#8947
Groenbech96 merged 3 commits into
mainfrom
user/magnushar/626501-permissions-overview-v2

Conversation

@Groenbech96

@Groenbech96 Groenbech96 commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Summary

Implements AB#626501 - Broader usage of the Permissions Overview page.

Adds navigation actions to open the Permissions Overview page (with optional filters) from:

  • Permission Sets list page (Where-Used + Permissions Overview actions)
  • Permission Set card page (Where-Used action)
  • Table Information page (View Table Permissions action)
  • Page Inspection page (View table permissions link)

System Application changes

  • New \Permissions Overview\ codeunit (9865) with integration events for cross-app navigation
  • Permission Set card: Added Where-Used action
  • Table Information page: Added View Table Permissions action
  • Table Information app.json: Added dependency on Permission Sets module
  • PermissionSetsObjects.PermissionSet.al: Registered new codeunit

BaseApp changes

  • PermissionsOverview.Page.al: Added SetInitialRoleIDFilter, SetInitialObjectFilter setters + ApplyInitialFilters on open
  • PermissionSets.Page.al: Added Where-Used (filtered) and Permissions Overview (unfiltered) actions
  • PermissionsOverviewHandler.Codeunit.al (new): Event subscriber bridging System App events to BaseApp page
  • PageInspection.Page.al: Added View table permissions drilldown link

Design

Uses integration events so BaseApp can subscribe and handle the actual page navigation, since the Permissions Overview page lives in BaseApp and System Application cannot reference it directly.

Add navigation actions to open the Permissions Overview page (with
optional filters) from multiple pages:

- Permission Sets list: Where-Used (filtered) + Permissions Overview
- Permission Set card: Where-Used (filtered)
- Table Information: View Table Permissions (filtered to table)
- Page Inspection: View Table Permissions link (filtered to table)

System Application changes:
- New Permissions Overview codeunit (9865) with integration events
- Where-Used action on Permission Set card
- View Table Permissions action on Table Information page
- Permission Sets dependency added to Table Information module

BaseApp changes:
- Permissions Overview page: setter procedures + ApplyInitialFilters
- Permission Sets list page: Where-Used + Permissions Overview actions
- New Permissions Overview Handler codeunit (9886) event subscriber
- Page Inspection: View table permissions drilldown link

Work Item: AB#626501

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@Groenbech96
Groenbech96 requested a review from a team July 1, 2026 07:04
@Groenbech96
Groenbech96 requested review from a team as code owners July 1, 2026 07:04
@github-actions github-actions Bot added this to the Version 29.0 milestone Jul 1, 2026
@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Copilot PR Review

Iteration 2 · Outcome: completed

Knowledge source: https://github.com/microsoft/BCQuality@822cae1b2771ac25f665f73369f69093bd4fd630

Orchestrator pre-filter (13 file(s) excluded)

  • layer-disabled (knowledge) : 13 file(s)

Findings produced by the Copilot CLI agent against BCQuality at 822cae1b2771ac25f665f73369f69093bd4fd630. Reply 👎 on any inline comment to flag false positives.

@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

⚠️ Stale Status Check Deleted

The Pull Request Build workflow run for this PR was older than 72 hours and has been deleted.

📋 Why was it deleted?

Status checks that are too old may no longer reflect the current state of the target branch. To ensure this PR is validated against the latest code and passes up-to-date checks, a fresh build is required.


🔄 How to trigger a new status check:

  1. 📤 Push a new commit to the PR branch, or
  2. 🔁 Close and reopen the PR

This will automatically trigger a new Pull Request Build workflow run.

@gggdttt gggdttt 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.

Can we add some tests for this new action? It should be testable right?

@Groenbech96
Groenbech96 enabled auto-merge July 13, 2026 10:11
@JesperSchulz JesperSchulz added the Integration GitHub request for Integration area label Jul 16, 2026
@Groenbech96

Copy link
Copy Markdown
Contributor Author

@copilot Pull latest main into the PR.

Copilot AI commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

@copilot Pull latest main into the PR.

Merged the latest main into this PR in cb9af5372.

@Groenbech96
Groenbech96 added this pull request to the merge queue Jul 20, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 20, 2026
@Groenbech96
Groenbech96 added this pull request to the merge queue Jul 20, 2026
Merged via the queue into main with commit cf133f1 Jul 20, 2026
484 of 491 checks passed
@Groenbech96
Groenbech96 deleted the user/magnushar/626501-permissions-overview-v2 branch July 20, 2026 20:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AL: System Application Integration GitHub request for Integration area

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants