Uploaded image for project: 'mod-orders'
  1. mod-orders
  2. MODORDERS-123

Orders and PO Lines endpoints restructure

    XMLWordPrintable

Details

    • Story
    • Status: Closed (View Workflow)
    • P3
    • Resolution: Done
    • None
    • 3.0.0
    • None
    • ACQ Sprint 55
    • 5
    • Thunderjet

    Description

      Currently the /orders/<id> endpoints do not allow to add endpoint like following because the <id> is string of specific pattern:
      /orders/receive
      /orders/check-in
      /orders/receiving-history
      /orders/po-number

      The approach is to rename existing endpoints to avoid naming collisions. Please refer to Orders API Listing for more details.

      Method Old Path New Path Request Response
      POST /orders /orders/composite-orders composite_purchase_order composite_purchase_order
      GET N/A /orders/composite-orders CQL query arg purchase_order_collection
      GET /orders/<id> /orders/composite-orders/<id> N/A composite_purchase_order
      PUT /orders/<id> /orders/composite-orders/<id> composite_purchase_order composite_purchase_order
      DELETE /orders/<id> /orders/composite-orders/<id> N/A 204
      POST /orders/<orderId>/lines /orders/order-lines composite_po_line composite_po_line
      GET N/A /orders/order-lines CQL query arg po_line_collection
      GET /orders/<orderId>/lines/<lineId> /orders/order-lines/<id> N/A composite_po_line
      PUT /orders/<orderId>/lines/<lineId> /orders/order-lines/<id> composite_po_line composite_po_line
      DELETE /orders/<orderId>/lines/<lineId> /orders/order-lines/<id> N/A 204

      Once this is done the following modules have to be updated to consume new endpoints:

      • UI orders app
      • mod-gobi

      Acceptance Criteria:

      • existing endpoints are redefined
      • new endpoints are implemented
      • unit tests are updated
      • api tests are updated

      TestRail: Results

        Attachments

          Issue Links

            Activity

              People

                cmcnally Craig McNally
                piotr_kalashuk Piotr Kalashuk
                Craig McNally Craig McNally
                Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                  Created:
                  Updated:
                  Resolved:

                  TestRail: Runs

                    TestRail: Cases