636017 Move CRM report action tooltips to report objects#9544
Open
V-CWurtzen wants to merge 2 commits into
Open
636017 Move CRM report action tooltips to report objects#9544V-CWurtzen wants to merge 2 commits into
V-CWurtzen wants to merge 2 commits into
Conversation
Move the report action ToolTips for 16 CRM (Marketing) reports onto the report objects and remove the now-duplicate ToolTip from the W1 page actions that run them (Marketing/segment/contact pages and role centers, plus the Administrator role center and Data Administration). Reports include Campaign - Details, Contact - Labels, the Create Conts. from Bank Accs./Customers/Vendors set, the Delete Campaign Entries / Interaction Log Entries / Logged Segments / Opportunities / Tasks set, Questionnaire - Handouts, Relocate Attachments, Sales Cycle - Analysis, Salesperson - Opportunities / Tasks, and Team - Tasks. All source tooltips are descriptive (no stubs).
Integrate the W1 CRM tooltip move into the forked country layers, as required by the Miapp sync gate (VerifyMiappSync): - 10 page forks across APAC, BE, CH, DACH, NA, NL, NO, RU: removed the duplicate ToolTip from the covered-report actions (33 lines). - 2 report forks (APAC ContactLabels, RU CreateContsfromVendors): added the W1 report-level ToolTip.
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 was referenced Jul 16, 2026
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 CRM / Marketing 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):
Campaign - Details,Contact - Labels,Create Conts. from Bank Accs.,Create Conts. from Customers,Create Conts. from Vendors,Delete Campaign Entries,Delete Interaction Log Entries,Delete Logged Segments,Delete Opportunities,Delete Tasks,Questionnaire - Handouts,Relocate Attachments,Sales Cycle - Analysis,Salesperson - Opportunities,Salesperson - Tasks,Team - Tasks. All source tooltips are descriptive (no stubs).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 (Marketing/segment/contact pages and role centers, plus the Administrator role center and Data Administration). 29 files.VerifyMiappSync):ToolTipremoved from the covered actions (33 lines).APAC ContactLabels,RU CreateContsfromVendors): report-levelToolTipadded to match W1.Total: 41 files (29 W1 + 12 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) and Bank (#9540) on the new BCApps pipeline.