ONDC-Official / v1.2.0-logs

Retail and Logistics Logs for 1.2.0
9 stars 270 forks source link

Chattybao (RET10) - Compliance check #150

Closed sahil-ondc closed 10 months ago

sahil-ondc commented 11 months ago

on_search_full_catalog_refresh

on_init

on_confirm

@venkat77777

venkat77777 commented 11 months ago

@sharmasahil0331 fixed the issues and submitted new PR, please check

venkat77777 commented 11 months ago

https://github.com/ONDC-Official/v1.2.0-logs/pull/154

venkat77777 commented 11 months ago

@sharmasahil0331 flow 4 and flow 5 logs are submitted for verification. Please check https://github.com/ONDC-Official/v1.2.0-logs/pull/171

sahil-ondc commented 11 months ago

on_search_full_catalog_refresh

on_select

venkat77777 commented 11 months ago

@sharmasahil0331 i have fixed above mentioned issues and have captured logs for all flows from flow 1 to flow 5 and updated the current PR.

Please check and share feedback

venkat77777 commented 11 months ago

@sharmasahil0331 added IGM & RSF logs from 1.1.0 logs repo to this repo with this PR https://github.com/ONDC-Official/v1.2.0-logs/pull/187 please check and share feedback

sahil-ondc commented 11 months ago

on_search_inc_refresh

init

on_status

@venkat77777

venkat77777 commented 11 months ago

@sharmasahil0331 if possible can you please review all the logs and share feedback at once?

venkat77777 commented 11 months ago

@sharmasahil0331 have made the feedback changes and submitted PR https://github.com/ONDC-Official/v1.2.0-logs/pull/214

sahil-ondc commented 11 months ago

on_search_inc_refresh

on_search_full_catalog_refresh

select

on_confirm

@venkat77777

sahil-ondc commented 10 months ago

Flow 1

search_full_catalog_refresh

on_search_full_catalog_refresh

search_inc_catalog_refresh

on_search_inc_catalog_refresh

Flow 2

on_select

Flow 4

on_cancel

Note

@venkat77777

BLR-0118 commented 10 months ago

Flow 1

  1. /on_search:

    • close_timing to be sent only if the store was previously sent as closed using full or incremental pull;
    • remove accept_bap_terms (as static terms aren't yet enabled);
    • for grocery packaged items (e.g. item ded21392-1a41-11ee-8ddd-2bac5af1417f:MlepomA3E7XUC_7MpLJgjn), pls also send EAN code in item.descriptor.code;
    • item unitized measure should show 10 kg for the above item;
    • pls use updated categories from this list;
  2. /on_search_incremental:

    • pls share logs for common grocery use cases such as item out of stock, change in price;

Flow 2

  1. /on_init:

    • remove provider_location, cancellation_terms;
  2. /on_confirm:

    • structure for bpp_terms is invalid;

Flow 5

  1. /on_cancel_rto:
    • for grocery, item costs are generally refunded in full and items are sent back to the store for resale; why not make your RTO fulfillment, quote trail & quote realistic?

for all flows with quote trail, pls check that there is no object with 0 value;

Flow 3, 4, 6 - ok;

pls resubmit logs only for issues identified above

BLR-0118 commented 10 months ago

@venkat77777

BLR-0118 commented 10 months ago

Flow 1

  1. /on_search:

    • item.quantity.available.count of "0" means out-of-stock & "99" means in-stock; similarly, item.quantity.maximum.count of "99" means no order cap and any other non-zero value means there's an order cap; so, what does max count of "0" for item e54e3fc0-944c-11ee-848d-2d2b8538311e:MR8mcezPPQEA-r8EhP6UN9 mean?
  2. /on_search (incremental):

    • for item "f862b941-1a41-11ee-8ddd-2bac5af1417f:MR8mcezPPQEA-r8EhP6UN9", how can maximum_value (MRP) change?
    • same with item "e3007b00-1a41-11ee-8ddd-2bac5af1417f:MR8mcezPPQEA-r8EhP6UN9";

Flow 2

  1. /on_init:

    • pls remove everything from payment except settlement_details, buyer finder fee (as these will be filled up by buyer app in case of prepaid payment collection by them);
  2. /on_confirm:

    • why is provider.locations.id null?
    • pls also add "provider_tax_number" to bpp_terms for the buyer NP to verify that you're an MSN;
BLR-0118 commented 10 months ago

@venkat77777 - pls resubmit logs only for issues above; also, need timeline by which item unitized quantity will be correctly filled up & when do you plan to enable variants?

venkat77777 commented 10 months ago

@BLR-0118 Have done the feedback changes and submitted the required logs in the PR https://github.com/ONDC-Official/v1.2.0-logs/pull/491

Please review and share your feedback.

Item Unitized quantity correction will be done by Feb 15th and regarding Variants, we are planning to do them by Feb 28th or F&B category inclusion whichever is earlier

BLR-0118 commented 10 months ago

@venkat77777 - Feb 15th for item unitized quantity correction is too late. I can have the QA process initiated (assuming your logs are ok) but you need to have this fixed within the next 2 weeks

venkat77777 commented 10 months ago

@BLR-0118 understand. We'll try to finish it within this month itself Supriyo. I'll update you as soon as it is done

BLR-0118 commented 10 months ago

following issues pending:

  1. setting up correct values for item unitized quantity (to be enabled within 2 weeks);
  2. enabling variants; logs cleared for grocery (RET10) subject to the above issues which will be fixed before moving to Prod;
BLR-0118 commented 10 months ago

@venkat77777