Skip to content

Add sell-to fields to PrintVis Production Analysis - #16

Draft
HenrikL359 with Copilot wants to merge 38 commits into
mainfrom
copilot/add-sell-to-columns-6027028
Draft

HenrikL359 with Copilot wants to merge 38 commits into
mainfrom
copilot/add-sell-to-columns-6027028

Conversation

Copilot AI commented Sep 21, 2026 •

Copy link
Copy Markdown
Contributor

PrintVis Production Analysis needs Sell-to No. and Sell-to Name derived from each line’s PrintVis Order No.. This change adds those fields and keeps them aligned with the related PVS Case record when a unique order match exists.

  • Schema

    • Add stored fields to PrintVis Production Analysis:
      • Sell-to No.
      • Sell-to Name
    • Mark both as non-editable customer content fields.
  • Population logic

    • Add shared sync logic that resolves PrintVis Order No. to PVS Case.
    • Copy PVS Case."Sell-To No." and PVS Case."Sell-To Name" only when the order number maps to exactly one case.
    • Leave the fields blank for missing or ambiguous order matches.
  • Write paths

    • Populate the fields for new analysis lines on insert.
    • Recalculate the fields when PrintVis Order No. is validated.
  • Existing data

    • Backfill historical PrintVis Production Analysis rows during install/upgrade.
    • Bump the app version so the upgrade path runs.

Example of the sync target:

SellToNo := PVSCase."Sell-To No.";
SellToName := PVSCase."Sell-To Name";

Copilot AI and others added 21 commits September 21, 2026 14:42
Co-authored-by: HenrikL359 <295621344+HenrikL359@users.noreply.github.com>
Co-authored-by: HenrikL359 <295621344+HenrikL359@users.noreply.github.com>
Co-authored-by: HenrikL359 <295621344+HenrikL359@users.noreply.github.com>
Co-authored-by: HenrikL359 <295621344+HenrikL359@users.noreply.github.com>
Co-authored-by: HenrikL359 <295621344+HenrikL359@users.noreply.github.com>
Co-authored-by: HenrikL359 <295621344+HenrikL359@users.noreply.github.com>
Co-authored-by: HenrikL359 <295621344+HenrikL359@users.noreply.github.com>
Co-authored-by: HenrikL359 <295621344+HenrikL359@users.noreply.github.com>
Co-authored-by: HenrikL359 <295621344+HenrikL359@users.noreply.github.com>
Co-authored-by: HenrikL359 <295621344+HenrikL359@users.noreply.github.com>
Co-authored-by: HenrikL359 <295621344+HenrikL359@users.noreply.github.com>
Co-authored-by: HenrikL359 <295621344+HenrikL359@users.noreply.github.com>
Co-authored-by: HenrikL359 <295621344+HenrikL359@users.noreply.github.com>
Co-authored-by: HenrikL359 <295621344+HenrikL359@users.noreply.github.com>
Co-authored-by: HenrikL359 <295621344+HenrikL359@users.noreply.github.com>
Co-authored-by: HenrikL359 <295621344+HenrikL359@users.noreply.github.com>
Co-authored-by: HenrikL359 <295621344+HenrikL359@users.noreply.github.com>
Co-authored-by: HenrikL359 <295621344+HenrikL359@users.noreply.github.com>
Co-authored-by: HenrikL359 <295621344+HenrikL359@users.noreply.github.com>
Co-authored-by: HenrikL359 <295621344+HenrikL359@users.noreply.github.com>
Co-authored-by: HenrikL359 <295621344+HenrikL359@users.noreply.github.com>
Copilot AI changed the title [WIP] Add Sell-to No. and Sell-to Name columns to PrintVis Production Analysis Add sell-to fields to PrintVis Production Analysis Sep 21, 2026
Copilot AI requested a review from HenrikL359 September 21, 2026 14:55
Co-authored-by: PropmanProphecy <8543071+PropmanProphecy@users.noreply.github.com>
Copilot AI and others added 7 commits September 22, 2026 06:41
Co-authored-by: PropmanProphecy <8543071+PropmanProphecy@users.noreply.github.com>
Co-authored-by: PropmanProphecy <8543071+PropmanProphecy@users.noreply.github.com>
Co-authored-by: PropmanProphecy <8543071+PropmanProphecy@users.noreply.github.com>
Co-authored-by: PropmanProphecy <8543071+PropmanProphecy@users.noreply.github.com>
Co-authored-by: PropmanProphecy <8543071+PropmanProphecy@users.noreply.github.com>
Co-authored-by: PropmanProphecy <8543071+PropmanProphecy@users.noreply.github.com>
Co-authored-by: PropmanProphecy <8543071+PropmanProphecy@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants