-
Notifications
You must be signed in to change notification settings - Fork 366
Logs for IGM 2.0 & RSF 2.0 #3602
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
Logs for IGM 2.0 & RSF 2.0 #3602
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In RSF Flow2
settle2,on_settle2 missing.
@ParneetkaurDev Please check now. I have updated rsf flow 2 logs. |
Flow 5 is mandatory for the Buyer; please ensure it is included. |
@ParneetkaurDev Please consider the latest commit. I have added rsf flow 5. |
Hi Parneet,
Please consider the latest commit, i added flow 5 for rsf.
Thanks
…On Thu, May 1, 2025 at 4:32 PM ParneetkaurDev ***@***.***> wrote:
*ParneetkaurDev* left a comment (ONDC-Official/v1.2.0-logs#3602)
<#3602 (comment)>
@adarsh-206 <https://github.com/adarsh-206>
Flow 5 is mandatory for the Buyer; please ensure it is included.
—
Reply to this email directly, view it on GitHub
<#3602 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/APYKEQ2M4AQ57VPGNQFZUQD24H5N5AVCNFSM6AAAAAB4DAZAEGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDQNBUGY2DQNZTGM>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
I reviewed the flow, and it should be Flow 5 (NIL flow). However, the response being sent corresponds to the NP-NP flow. Please review and correct this as per the mock server guide. Kindly ensure all flows align with the definitions provided in the document. |
@ParneetkaurDev Please consider the latest commit again. I have changes logs for Flow 5 (NIL flow). The other report, on_report, recon, and on_recon were showing errors in utility, I guess those are not applicable in case of NIL. Please review this. |
No description provided.