Skip to content

636017 Move Manufacturing report action tooltips to report objects#9580

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

636017 Move Manufacturing report action tooltips to report objects#9580
V-CWurtzen wants to merge 2 commits into
mainfrom
features/636017-Manufacturing-move-page-action-tooltips-to-report

Conversation

@V-CWurtzen

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

Copy link
Copy Markdown

What & why

Moves the report action ToolTips for 7 Manufacturing reports off the page actions that run them and onto the report objects themselves. The tooltip is now defined once on the report and applies wherever the report is invoked — the page action inherits it via the 2025 release wave 1 feature "Running objects in actions defaults to UI descriptors on target object" — and the now-duplicate ToolTip is removed from the page actions.

Reports moved: Capacity Task List, Inventory Valuation - WIP, Prod. Order - Shortage List, Production Order Statistics, Quantity Explosion of BOM, Subcontractor - Dispatch List, Where-Used (Top Level).

This completes the Manufacturing area — the pre-migration ADO pass was partial (IT-only, 4 files) and, via a since-fixed tooling bug, skipped reports that have obsolete dataset columns. classify now reports Manufacturing = 0 remaining.

Scope

  • W1 (src/Layers/W1/BaseApp/Manufacturing): 22 files — 7 report-level ToolTip adds + 27 duplicate action ToolTip removals across 15 pages/role centers.
  • Country: the IT layer is the only fork of these pages — 5 IT forks updated (12 removals). No other country forks exist.

Metadata-only, tooltip-only diff; BOM/EOL preserved.

Fixes AB#636017

Completes the Manufacturing W1 move (the pre-GitHub ADO pass was partial and,
via the now-fixed scan-tooltips obsolete-column bug, skipped reports with
obsolete columns). Moves 7 reports -- Capacity Task List, Inventory Valuation
- WIP, Prod. Order - Shortage List, Production Order Statistics, Quantity
Explosion of BOM, Subcontractor - Dispatch List, Where-Used (Top Level) --
adding report-level ToolTips and removing the 27 duplicate action ToolTips
across 15 pages/role centers.

classify now reports Manufacturing = 0 RealMiss / 0 Stub / 0 ResidualDup
(remaining 15 are intentional obsolete skips). Tooltip-only; BOM/EOL preserved.
Country propagation for the Manufacturing W1 move. Only the IT layer forks
these pages, so 5 IT forks are updated (12 duplicate action ToolTips removed
for the covered reports); no other country forks exist, and the 7 reports
have no country forks. Fork set derived with Invoke-Miapp (authoritative);
changed set exactly matches (0 missing / 0 extra). Tooltip-only; BOM/EOL
preserved.
@V-CWurtzen
V-CWurtzen requested a review from a team July 17, 2026 13:43
@github-actions github-actions Bot added Other GitHub request for other area than SCM, Finance or Integration Ownership: Needs Review Ownership is Other, low confidence, or needs manual correction labels Jul 17, 2026
@github-actions github-actions Bot added this to the Version 29.0 milestone Jul 17, 2026
DefaultRenderingLayout = ExcelLayout;
ApplicationArea = Manufacturing;
Caption = 'Capacity Task List';
ToolTip = 'View the production orders that are waiting to be processed at the work centers and machine centers. Printouts are made for the capacity of the work center or machine center). The report includes information such as starting and ending time, date per production order and input quantity.';

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.

$\textbf{🟡\ Medium\ Severity\ —\ Accessibility} \quad \color{gray}{\texttt{\small Iteration\ 1}}$

The canonical ToolTip added to CapacityTaskList.Report.al carries a pre-existing stray closing parenthesis: '...capacity of the work center or machine center).

The report includes...'. Shop Supervisor Role Center's own 'Capacity Task List' action (both the IT and W1 forks) previously had the corrected wording without the stray ')'. Removing that page-level ToolTip in favor of the report-level one reintroduces the typo on Shop Supervisor Role Center, where it had been fixed. Since the report ToolTip is now the single source of truth for every page that runs this report, fixing the typo there benefits all callers.

Suggested change
ToolTip = 'View the production orders that are waiting to be processed at the work centers and machine centers. Printouts are made for the capacity of the work center or machine center). The report includes information such as starting and ending time, date per production order and input quantity.';
ToolTip = 'View the production orders that are waiting to be processed at the work centers and machine centers. Printouts are made for the capacity of the work center or machine center. The report includes information such as starting and ending time, date per production order and input quantity.';

👍 useful · ❤️ especially valuable · 👎 wrong - reply with why

Caption = 'Production Order Statistics';
Image = "Report";
RunObject = report "Production Order Statistics";
ToolTip = 'View statistical information, such as the value of posted entries, for the record.';

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.

$\textbf{🟡\ Medium\ Severity\ —\ Accessibility} \quad \color{gray}{\texttt{\small Iteration\ 1}}$

This PR is described as removing duplicate ToolTip properties in favor of a single canonical ToolTip on the target report, relying on the 2025 wave 1 action-inherits-target-tooltip feature.

For most actions the removed page-level text is an exact match of the new report-level text, but for the 'Production Order Statistics' action on Production Planner Role Center, Shop Supervisor Mfg Foundation, and Shop Supervisor Role Center (both the IT and W1 forks, 6 files total) the removed text was 'View statistical information, such as the value of posted entries, for the record.' -- materially different wording from the canonical text now added to ProductionOrderStatistics.Report.al ('View statistical information about the production order's direct material and capacity costs and overhead as well as its capacity need in the time unit of measure.'). End users of those three role centers will now see different, longer tooltip text than before, and translators will need to re-localize the changed string. Confirm this wording change is intentional (not just a de-duplication) before merging, or restore page-specific ToolTips on those three actions if the shorter wording was deliberate.

👍 useful · ❤️ especially valuable · 👎 wrong - reply with why

@github-actions

Copy link
Copy Markdown
Contributor

Copilot PR Review

Iteration 1 · Outcome: completed

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

Findings by domain

Findings split into Knowledge-backed (cite a BCQuality article) and Agent (the agent's own judgement, no matching BCQuality rule).

Domain Findings Knowledge-backed Agent Inline Fallback
Accessibility 2 2 0 2 0

Totals: 2 knowledge-backed · 0 agent findings.

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: Needs Review Ownership is Other, low confidence, or needs manual correction

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant