Skip to content
This repository has been archived by the owner on Jul 4, 2024. It is now read-only.

Shiprocket - compliance check #32

Closed
BLR-0118 opened this issue Mar 9, 2023 · 7 comments
Closed

Shiprocket - compliance check #32

BLR-0118 opened this issue Mar 9, 2023 · 7 comments

Comments

@BLR-0118
Copy link
Member

BLR-0118 commented Mar 9, 2023

  1. /on_confirm - order.created_at can't be changed as it was created in /confirm;
  2. /on_update - shipping label pdf should be sent in fulfillment.start.instructions.images;
  3. /on_status (pickup) - fulfillment.start.time.timestamp is mandatory;
  4. /on_status (delivered) - fulfillment.end.time.timestamp is mandatory;
  5. /on_cancel - fulfillment.id should be same as in /on_confirm but fulfillment state code should be set to cancelled;
@BLR-0118
Copy link
Member Author

BLR-0118 commented Mar 9, 2023

@pankajdubey

@pankajd16
Copy link

@BLR-0118 Will check and update you.

@pankajd16
Copy link

pankajd16 commented Mar 14, 2023

@BLR-0118 Fixes pushed in the PR below please review.
#73

@BLR-0118
Copy link
Member Author

Logs with Sellerapp

  1. /on_init:
  • provider id "60" & item id "60" selected in /init (for which quote was 2 in /on_search) but here the quote shows as 121.8 (which is the corresponding RTO quote);

@BLR-0118
Copy link
Member Author

@pankajd16

@pankajd16
Copy link

pankajd16 commented Mar 15, 2023

@BLR-0118

  1. /search:
    The start and end pincodes are 560038 and 500081 resp. 560038 is Bangalore and 500081 is Hyderabad so, in this case, the Shipment zone is Zone_d. As this is a test environment so the zone rate for this zone is Rs 2. (Dummy Rate)

  2. /init:
    The start and end pincodes are 560038 and 560038 resp. 560038 is Bangalore only so, in this case, the Shipment zone is Zone_a. And for Zone_a in the database we have a zone rate of Rs 121.80 (Dummy Rate).

There was a change in pincodes in both requests and rates will change accordingly. Please let me know in case of any concerns.

@BLR-0118
Copy link
Member Author

Ok. Logistics buyer app should have called /search again but that's not happening in this case. Anyway, these logs are cleared for v1.1.0

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants