Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Costbo LBNP (RSF) - compliance check #2682

Open
JyotiSharma0711 opened this issue Oct 23, 2024 · 6 comments
Open

Costbo LBNP (RSF) - compliance check #2682

JyotiSharma0711 opened this issue Oct 23, 2024 · 6 comments

Comments

@JyotiSharma0711
Copy link

**Costbo(LBNP)

flow 1

happy flow & Correction flow

receiver_recon**

  • "/message/orderbook/orders/4/payment/params/amount must be number",
  • "/message/orderbook/orders/4/payment/@ondc1org1buyer_app_finder_fee_amount must be number",
  • "/message/orderbook/orders/4/payment/@ondc1org1buyer_app_finder_fee_amount must be number",
  • "/message/orderbook/orders/4/withholding_tax_gst/value must be number",
  • message/orderbook/orders/4/withholding_tax_tds/value must be number",
  • message/orderbook/orders/4/deduction_by_collector/value must be number",
  • "/message/orderbook/orders/4/payerdetails/payer_account_no must be string",
  • context timestamp should be greater than updated_at"

on_receiver_recon

  • "/message/orderbook/orders/0/counterparty_diff_amount/value must be number"

@AnanthBo

@AnanthBo
Copy link
Contributor

@JyotiSharma0711 , The documentation here has a different from what your comments have. {{ 1. . strings versus numbers. 2. context.timestamp can be same (equal) or higher than updated_at. Please check again. }}.
Document: https://docs.google.com/document/d/1ubUPAWpbbUJ4vG2h5TQ74srZBjYjrO0P/edit

Can we connect for a meeting ? Which document are you referring ? Why is there a disparity ? I am reachable at 9845570864.

@AnanthBo
Copy link
Contributor

AnanthBo commented Oct 24, 2024

@JyotiSharma0711 , Please connect to understand the difference or. correct RSF documentation. can you please call at my number for clarifications

@AnanthBo
Copy link
Contributor

@JyotiSharma0711 Fixed and pushed logs PR 2689. Please approve at the earliest. as the changes is extremely minimal .
with #2689

@AnanthBo
Copy link
Contributor

@JyotiSharma0711 can you please share the update on this

@AnanthBo
Copy link
Contributor

@JyotiSharma0711 can you please share the update on this

@RupalSingla
Copy link

logs approved @AnanthBo

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

No branches or pull requests

3 participants