Skip to content

[Purchases] Use posted return-shipment report selection for PDF - #11230

Draft
Prangshuman Das (t-prda) wants to merge 13 commits into
prdas/646383-split-field-comparisonfrom
prdas/646383-split-return-shipment-pdf
Draft

[Purchases] Use posted return-shipment report selection for PDF#11230
Prangshuman Das (t-prda) wants to merge 13 commits into
prdas/646383-split-field-comparisonfrom
prdas/646383-split-return-shipment-pdf

Conversation

@t-prda

@t-prda Prangshuman Das (t-prda) commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Scope

Change Return Shpt. PDF Doc.Handler.GeneratePdfBlobWithDocumentType from P.Return to P.Ret.Shpt. This is the only production-code change in the seven-way split and affects report selection/attachment naming.

AB#646383 — follow-up split from PR #10085; link only, not an instruction to resolve the umbrella work item.

Evidence and limits

No corresponding PDF failure appears in saved result sets 2 or 4. The existing APIV2 purchase-return-shipment suite exclusion is unchanged. A targeted posted-return-shipment PDF regression run is still required.

Native GitHub stack #11232

This draft is stacked above #11229. #10085 is the bottom API-authentication fix. The review diff contains only this layer's fix plus removal of its 0 temporary method exclusions. Earlier layers are inherited, not repeated in this diff.

Existing PR/branch history is preserved; no force-push or merge into main was performed. NAV selection/company/native exclusions and existing out-of-scope Expense exclusions remain untouched.

Validation

  • Parent-tip ancestry and exact layer file scope verified; git diff whitespace checks and exclusion JSON validation passed.
  • Only this layer's listed temporary exclusions are removed; existing whole-codeunit exclusions are retained.
  • Full runtime validation of this stacked head is pending. Historical failures are issue evidence, not proof this head passes.
  • Repository CI currently filters PR base branches to main, releases/* and features/*; retargeted personal-branch layers do not automatically trigger that workflow on subsequent pushes. No workflow-policy expansion was made.

Methods re-enabled by this layer

None added. The existing purchase-return-shipment suite exclusion remains unchanged.

Split from the preserved PR10085 snapshot. AB#646383

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Preserve published branch history and re-enable only the 0 methods covered by this layer.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 3952f078-a881-4da8-ad96-13b727e48a91
@t-prda
Prangshuman Das (t-prda) changed the base branch from main to prdas/646383-split-field-comparison September 8, 2026 17:06
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 3952f078-a881-4da8-ad96-13b727e48a91
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 3952f078-a881-4da8-ad96-13b727e48a91
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 3952f078-a881-4da8-ad96-13b727e48a91
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 3952f078-a881-4da8-ad96-13b727e48a91
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 3952f078-a881-4da8-ad96-13b727e48a91
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 3952f078-a881-4da8-ad96-13b727e48a91
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 3952f078-a881-4da8-ad96-13b727e48a91
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 3952f078-a881-4da8-ad96-13b727e48a91
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 3952f078-a881-4da8-ad96-13b727e48a91
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 3952f078-a881-4da8-ad96-13b727e48a91
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 3952f078-a881-4da8-ad96-13b727e48a91
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Team: SCM GitHub request for SCM area

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant