Skip to content

Issue#7867: Removed the access restriction for procedures in the confirmation window.#8871

Open
arvindbinarystream wants to merge 1 commit into
microsoft:mainfrom
arvindbinarystream:Issues#7867
Open

Issue#7867: Removed the access restriction for procedures in the confirmation window.#8871
arvindbinarystream wants to merge 1 commit into
microsoft:mainfrom
arvindbinarystream:Issues#7867

Conversation

@arvindbinarystream

@arvindbinarystream arvindbinarystream commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

What & why

This change allows us to use the confirmation window without any restrictions, eliminating the need to add a copy object to our app. Since the page has no source table and is a standard dialog type, the update is minor and does not introduce any significant impact.

Linked work

Fixes #7867

For this issue, we also needed to add a few procedures to the facade codeunit, but I didn’t include those changes since another PR from @Franco111000 is already open: #8713

How I validated this

  • I read the full diff and it contains only changes I intended.
  • I built the affected app(s) locally with no new analyzer warnings.
  • I ran the change in Business Central and confirmed it behaves as expected.
  • I added or updated tests for the new behavior, or explained below why none are needed.

What I tested and the outcome (required — be specific: scenarios, commands, screenshots for UI changes)

Risk & compatibility

@arvindbinarystream arvindbinarystream requested a review from a team June 29, 2026 19:47
@github-actions github-actions Bot added AL: Apps (W1) Add-on apps for W1 From Fork Pull request is coming from a fork needs-approval Workflow runs require maintainer approval to start labels Jun 29, 2026
@JesperSchulz JesperSchulz added the Integration GitHub request for Integration area label Jun 30, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Copilot PR Review

Iteration 1 · 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 removed the needs-approval Workflow runs require maintainer approval to start label Jun 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AL: Apps (W1) Add-on apps for W1 From Fork Pull request is coming from a fork Integration GitHub request for Integration area

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BC Idea]: Add requested procedures in facade codeunit to match BC behavior.

2 participants