ONDC-Official/v1.1.0-logs

Bizom - compliance check

Closed this issue · 10 comments

BLR-0118 commented
  1. /on_search:
  • time.timestamp for "enable" needs to match Context.timestamp;
  • time.schedule.holidays is mandatory key (can be empty array);
  • time.range.end of "2400" is invalid time (can be "2359");
  1. /on_status:
  • pickup time (fulfillment.start.time.timestamp) and delivery time (fulfillment.end.time.timestamp) are mandatory as per the contract;
  1. /on_update1:
  • quote adjustment & refund doesn't happen in "return initiated" state;
  1. /on_update2:
  • quote adjustment & refund doesn't happen in "return approved" state;
samarthpatel-bizom commented

For "quote adjustment & refund doesn't happen in "return initiated" state":
-> Does that mean we are supposed to send the same quote including the items under return_initiated/ return_approved state ? And if the return is liquidated we can send the updated quote which BAP will consume and process refund ?

@BLR-0118 can you please validate whether my understanding is correct or not ?

samarthpatel-bizom commented

Also please clarify whether all the flows needed to be resubmitted or a single flow will be enough ?

BLR-0118 commented

There are specific states set by the seller at which the buyer app will trigger the refund (this is clearly documented in the API contract doc). It's only in these states that the quote adjustment should happen.

samarthpatel-bizom commented

Yes understood and have raised the PR accordingly: #121
Thanks

BLR-0118 commented
  1. /on_search:
  • contact_details_consumer_care should be in this format: "name,email,contactno";

Flow 1

  1. /on_status with an earlier timestamp than /on_update2 (which sets return state to "Liquidated") shows the same return state. How?
  2. /on_update2:
  • message_id should be same as /update (temporarily) as currently there's no way of correlating individual return requests;
samarthpatel-bizom commented
  1. Acknowledged
  2. The on_status was of the older order for which we were not sharing timestamp.
  3. Acknowledged: reusing same message id from /update.

Thanks.

@BLR-0118

BLR-0118 commented

logs cleared for v.1.1.0