Skip to content

Uptake/approval apis#29778

Open
BenPlunk wants to merge 10 commits intomicrosoft:mainfrom
BenPlunk:uptake/approval-apis
Open

Uptake/approval apis#29778
BenPlunk wants to merge 10 commits intomicrosoft:mainfrom
BenPlunk:uptake/approval-apis

Conversation

@BenPlunk
Copy link
Copy Markdown

@BenPlunk BenPlunk commented Mar 3, 2026

Summary

Added the following APIs:

Read-only API that allows to query Workflow
Read-only API that allows to query (Workflow, User) mappings
Read-only API that allows to query (Workflow, User, action) history

Depends on Objects in Base App PR https://github.com/microsoft/BusinessCentralApps/pull/1733

Work Item(s)

Fixes #29777
AB#613827

BenPlunk and others added 3 commits March 3, 2026 16:15
@BenPlunk BenPlunk requested a review from a team as a code owner March 3, 2026 07:33
@BenPlunk BenPlunk requested a review from mynjj March 3, 2026 07:33
@stkillen
Copy link
Copy Markdown
Contributor

stkillen commented Mar 3, 2026

Comment thread Apps/W1/APIV2/app/src/pages/APIV2ApprovalEntries.Page.al Outdated
Comment thread Apps/W1/APIV2/app/src/pages/APIV2ApprovalEntries.Page.al Outdated
Comment thread Apps/W1/APIV2/app/src/pages/APIV2ApprovalEntries.Page.al Outdated
Comment thread Apps/W1/APIV2/app/src/pages/APIV2ApprovalEntries.Page.al Outdated
Comment thread Apps/W1/APIV2/app/src/pages/APIV2ApprovalEntries.Page.al Outdated
Comment thread Apps/W1/APIV2/app/src/pages/APIV2ApprovalEntries.Page.al Outdated

page 30094 "APIV2 - Approval Entries"
{
APIGroup = 'auditing';
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.

If these are going to be a separate group, then they shouldn't be in APIV2 app. Instead they should get their own app.

{
APIGroup = 'auditing';
APIPublisher = 'microsoft';
APIVersion = 'v2.0';
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.

microsoft/auditing is a new API path, why v2.0?

  • Either no new path and part of standard APIs (not favorable)
  • Or new app (favorable)

BenPlunk and others added 2 commits March 4, 2026 10:49
Co-authored-by: Onat Buyukakkus <55088871+onbuyuka@users.noreply.github.com>
Co-authored-by: Onat Buyukakkus <55088871+onbuyuka@users.noreply.github.com>
@github-actions github-actions bot added the linked Issue is linked to a Azure Boards work item label Mar 3, 2026
Comment thread Apps/W1/APIV2/app/src/pages/APIV2ApprovalUserSetup.Page.al Outdated
Comment thread Apps/W1/APIV2/app/src/pages/APIV2PstdApprovalEntries.Page.al Outdated
Comment thread Apps/W1/APIV2/app/src/pages/APIV2PstdApprovalEntries.Page.al Outdated
Comment thread Apps/W1/APIV2/app/src/pages/APIV2ApprovalEntries.Page.al Outdated
Comment thread Apps/W1/APIV2/app/src/pages/APIV2PstdApprovalEntries.Page.al
Comment thread Apps/W1/APIV2/test/src/APIV2ApprovalEntriesE2E.Codeunit.al
Comment thread Apps/W1/APIV2/test/src/APIV2ApprovalEntriesE2E.Codeunit.al
Comment thread Apps/W1/APIV2/test/src/APIV2ApprovalEntriesE2E.Codeunit.al
@JesperSchulz JesperSchulz added the Integration GitHub request for Integration area label Mar 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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]: Auditing Reports - Approval APIs

5 participants