636017 Move Cost Accounting report action tooltips to report objects#9545
Open
V-CWurtzen wants to merge 2 commits into
Open
636017 Move Cost Accounting report action tooltips to report objects#9545V-CWurtzen wants to merge 2 commits into
V-CWurtzen wants to merge 2 commits into
Conversation
…(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.
Contributor
Copilot PR ReviewIteration 1 · Outcome: completed Knowledge source: https://github.com/microsoft/BCQuality@186d8a131465475c79244d994acb872cd5c0d4bf Orchestrator pre-filter (2 file(s) excluded)
Findings produced by the AL review agent v1.7.3. Reply 👎 on any inline comment to flag false positives. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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
ToolTipis 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
ToolTipadded 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.VerifyMiappSync):ToolTipremoved from the covered actions (232 lines).Total: 72 files (32 W1 + 40 country), tooltip-only.
Linked work
Fixes AB#636017
How I validated this
What I tested and the outcome
Metadata-only change (
ToolTipproperty). The diff is 100% tooltip lines (reportToolTipadditions + page-actionToolTipremovals), with BOM/EOL preserved and no other lines changed. Country propagation was produced by runningInvoke-Miappto 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 whatVerifyMiappSyncrequires — no missing forks, no extras.No tests added:
ToolTipis 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
ToolTipdesign 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.