Skip to content

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

Merged
onbuyuka merged 1 commit into
microsoft:mainfrom
arvindbinarystream:Issues#7867
Jul 10, 2026
Merged

Issue#7867: Removed the access restriction for procedures in the confirmation window.#8871
onbuyuka merged 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

Fixes AB#641602

@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
@JesperSchulz JesperSchulz added the Linked Issue is linked to a Azure Boards work item label Jul 7, 2026
@github-actions github-actions Bot added this to the Version 29.0 milestone Jul 7, 2026
@onbuyuka onbuyuka enabled auto-merge July 9, 2026 13:03
@onbuyuka onbuyuka added this pull request to the merge queue Jul 10, 2026
Merged via the queue into microsoft:main with commit aba239d Jul 10, 2026
481 of 492 checks passed
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 Linked Issue is linked to a Azure Boards work item

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.

3 participants