Skip to content

636017 Move Cost Accounting report action tooltips to report objects#9545

Open
V-CWurtzen wants to merge 2 commits into
mainfrom
features/636017-CostAccounting-move-page-action-tooltips-to-report
Open

636017 Move Cost Accounting report action tooltips to report objects#9545
V-CWurtzen wants to merge 2 commits into
mainfrom
features/636017-CostAccounting-move-page-action-tooltips-to-report

Conversation

@V-CWurtzen

@V-CWurtzen V-CWurtzen commented Jul 16, 2026

Copy link
Copy Markdown

What & why

Moves the report action ToolTips for the Cost Accounting reports off the page actions that run them and onto the report objects themselves, and integrates that move into every forked country layer. The tooltip is now defined once on the report and applies wherever the report is invoked, and the now-duplicate ToolTip is removed from the page actions in W1 and its country forks.

Reports covered (16): Cost Acctg. Analysis, Cost Acctg. Balance/Budget, Cost Acctg. Journal, Cost Acctg. Statement, Cost Acctg. Statement/Budget, Cost Acctg. Stmt. per Period, Cost Allocation, Cost Allocations, Cost Types Details, Copy Cost Budget, Copy Cost Acctg. Budget to G/L, Copy G/L Budget to Cost Acctg., Transfer Budget to Actual, Delete Cost Budget Entries, Delete Cost Entries, Delete Old Cost Entries.

Where only a Run the X report. stub existed (Cost Acctg. Journal), it is hoisted as the sole option — happy to replace with better wording if a reviewer suggests it.

This is part of the "move report action tooltips to report objects" cleanup (CP0529-331 / slice 636017).

Changes

  • W1 (commit 1): report-level ToolTip added to the 16 reports; duplicate removed from the W1 actions that run them (Cost Accounting pages/journals and role centers, plus the Accountant role center and Data Administration). 32 files.
  • Country layers (commit 2): the W1 move is propagated to the forked country layers as required by the Miapp sync gate (VerifyMiappSync):
    • 40 page forks across 18 country layers (APAC, AT, AU, BE, CH, DACH, DE, ES, FI, FR, GB, IS, IT, NL, NO, NZ, RU, SE): duplicate ToolTip removed from the covered actions (232 lines).
    • No report forks and no no-op touches were required for this area.

Total: 72 files (32 W1 + 40 country), tooltip-only.

Linked work

Fixes AB#636017

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

Metadata-only change (ToolTip property). The diff is 100% tooltip lines (report ToolTip additions + page-action ToolTip removals), with BOM/EOL preserved and no other lines changed. Country propagation was produced by running Invoke-Miapp to identify the fork set, then applying the move surgically (Invoke-Miapp's auto-merge mis-aligned on the divergent role centers, so its output was not used directly). The set of changed forks matches exactly what VerifyMiappSync requires — no missing forks, no extras.

No tests added: ToolTip is a design-time metadata property with no runtime behavior to assert; there is no existing tooltip test harness for these objects.

Risk & compatibility

Low. No code, schema, table, or behavior change — only the ToolTip design property moves from page action to report, in W1 and its country forks. No upgrade/data impact, no permission/telemetry impact, no breaking change. Same cleanup pattern already merged for Warehouse, Manufacturing, Service, Sales and Inventory, landed as the Pricing pilot (#9433), and applied to CashFlow (#9528), Finance (#9526), Bank (#9540) and CRM (#9544) on the new BCApps pipeline.

…(W1)

Move the report action ToolTips for 16 Cost Accounting reports onto the
report objects and remove the now-duplicate ToolTip from the W1 page
actions that run them (Cost Accounting pages/journals and role centers,
plus the Accountant role center and Data Administration).

Reports: the Cost Acctg. Analysis / Balance-Budget / Journal / Statement /
Statement-Budget / Stmt. per Period set, Cost Allocation(s), Cost Types
Details, the Copy Cost Budget / Copy Cost Acctg. Budget to G/L / Copy G/L
Budget to Cost Acctg. / Transfer Budget to Actual set, and the Delete Cost
Budget Entries / Cost Entries / Old Cost Entries set. Where only a
"Run the X report." stub existed (Cost Acctg. Journal) it is hoisted as
the sole option.
Integrate the W1 Cost Accounting tooltip move into the forked country
layers, as required by the Miapp sync gate (VerifyMiappSync):
- 40 page forks across 18 country layers (APAC, AT, AU, BE, CH, DACH, DE,
  ES, FI, FR, GB, IS, IT, NL, NO, NZ, RU, SE): removed the duplicate
  ToolTip from the covered-report actions (232 lines).

No report forks and no no-op touches were required for this area.

Ran Invoke-Miapp to identify the fork set; applied surgically (its
auto-merge mis-aligns on the divergent role centers). The changed-fork set
matches VerifyMiappSync's required set exactly (40 files). Metadata-only.
@V-CWurtzen
V-CWurtzen requested a review from a team July 16, 2026 16:08
@github-actions github-actions Bot added this to the Version 29.0 milestone Jul 16, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Copilot PR Review

Iteration 1 · Outcome: completed

Knowledge source: https://github.com/microsoft/BCQuality@186d8a131465475c79244d994acb872cd5c0d4bf

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

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

Findings produced by the AL review agent v1.7.3. Reply 👎 on any inline comment to flag false positives.

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

Labels

Other GitHub request for other area than SCM, Finance or Integration Ownership: Manual Preserve the manually selected team ownership

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants