Skip to content

[18.0][ADD] purchase_order_etd_eta#2941

Open
AungKoKoLin1997 wants to merge 1 commit intoOCA:18.0from
qrtl:18.0-add-purchase_order_etd_eta
Open

[18.0][ADD] purchase_order_etd_eta#2941
AungKoKoLin1997 wants to merge 1 commit intoOCA:18.0from
qrtl:18.0-add-purchase_order_etd_eta

Conversation

@AungKoKoLin1997
Copy link
Contributor

The module adds ETD, ETA, and Shipping Schedule Note fields to purchase orders and displays them in the purchase order and RFQ reports.

@qrtl QT6348

Copy link
Member

@yostashiro yostashiro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add CONTEXT.md to explain in what situations the provided functionality is useful.

Copy link
Contributor

@nobuQuartile nobuQuartile left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It’s a small detail, but I’d appreciate it if you could take a look.

Functional test: Done
Purchase Order Fields
Confirmed that ETD, ETA, and Shipping Schedule Note can be set and saved on Purchase Orders.
Search and Filters
Verified search, date filters, and group by for ETD, ETA, and Shipping Schedule Note.
List Views
Checked that ETD, ETA, and Shipping Schedule Note are available in list and KPI views.
Reports
Verified ETD, ETA, and Shipping Schedule Note appear correctly in RFQ and PO reports.
Expected Arrival Setting
Confirmed Expected Arrival is hidden in reports when the setting is enabled and ETA is set.

@AungKoKoLin1997 AungKoKoLin1997 force-pushed the 18.0-add-purchase_order_etd_eta branch 2 times, most recently from fed2ceb to 31bfba0 Compare January 29, 2026 02:49
Copy link
Member

@yostashiro yostashiro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some minor adjustments.

@AungKoKoLin1997 AungKoKoLin1997 force-pushed the 18.0-add-purchase_order_etd_eta branch from 31bfba0 to c3c00a0 Compare February 12, 2026 02:43
Copy link
Member

@yostashiro yostashiro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. 👍

Copy link

@kanda999 kanda999 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Functioanl review.

ETD and ETA are displayed in purchase report.

Image

Copy link
Contributor

@nobuQuartile nobuQuartile left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Functional review and code review: LGTM

@AungKoKoLin1997 AungKoKoLin1997 force-pushed the 18.0-add-purchase_order_etd_eta branch from c3c00a0 to 2ba8006 Compare February 17, 2026 08:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants