Skip to content

MODORDSTOR-535 - Update acq-models for PO line lastExport and export transmission method - #1356

Merged
markusweigelt merged 1 commit into
masterfrom
MODORDSTOR-535
Oct 12, 2026
Merged

markusweigelt merged 1 commit into
masterfrom
MODORDSTOR-535

Conversation

@markusweigelt

@markusweigelt markusweigelt commented Oct 9, 2026 •

Copy link
Copy Markdown
Contributor

MODORDSTOR-535 - Update acq-models for PO line lastExport and export transmission method

Purpose

After an order is exported via EDIFACT, its PO line cannot be opened. mod-orders-storage#653 (MODORDSTOR-526) started storing lastExport on PO lines and exportTransmissionMethod on export history (acq-models#580), but mod-orders still used the old schemas. Its generated PoLine and ExportHistory models reject unknown fields, so reading an exported line or its export history fails with Unrecognized field "lastExport" / Unrecognized field "exportTransmissionMethod".

Approach

  • acq-models submodule 64a23ab → 62fc7d4. This generates LastExport, PoLine.lastExport and ExportHistory.exportTransmissionMethod. - ModuleDescriptor:
    • Required interfaces: orders-storage.po-lines 13.0 → 13.1 and orders-storage.export-history 1.0 → 1.1, matching mod-orders-storage#653.
    • Provided interfaces: orders 13.0 → 13.1 and orders.export-history 1.0 → 1.1, because their responses now include the new optional fields.
  • NEWS.md: Bug fixes entry.

Reading exported PO lines and export history failed with "Unrecognized
field" because mod-orders still used outdated acq-models schemas.
Bump the required storage interfaces to po-lines 13.1 / export-history
1.1 and the provided orders 13.1 / orders.export-history 1.1.
@markusweigelt
markusweigelt requested a review from a team as a code owner October 9, 2026 15:19
@sonarqubecloud

sonarqubecloud Bot commented Oct 9, 2026

Copy link
Copy Markdown

@markusweigelt
markusweigelt merged commit c65bc54 into master Oct 12, 2026
20 checks passed
@markusweigelt
markusweigelt deleted the MODORDSTOR-535 branch October 12, 2026 07:48
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.

2 participants