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

Updated Investment logs #1803

Open
wants to merge 15 commits into
base: main
Choose a base branch
from

Conversation

Abhishekkumar86454
Copy link

No description provided.

@sourabhsaini11
Copy link

@Abhishekkumar86454, why are logs for both FIS12 & FIS14 present in same PR?, please segregate the flows based on domains into separate PR

@Abhishekkumar86454
Copy link
Author

@Abhishekkumar86454, why are logs for both FIS12 & FIS14 present in same PR?, please segregate the flows based on domains into separate PR

Hi @sourabhsaini11, I have removed FIS12 from this PR. Please verify it

@anuj-ondc
Copy link

@Abhishekkumar86454, Please fix below issues:
common issues:

  • created_at date is different from on_init call.

Please submit these flows also. (ref: log-submission guide)

  • search full pull
  • search Incremental-Pull
  • sip instalment
  • sip instalment Payment Failure
  • lumpsum payment retry (both payment failure and retries)
  • redemption failure

@92shreyansh 92shreyansh added the FIS-14 Investments label Nov 27, 2024
@Abhishekkumar86454
Copy link
Author

@Abhishekkumar86454, Please fix below issues: common issues:

  • created_at date is different from on_init call.

Please submit these flows also. (ref: log-submission guide)

  • search full pull
  • search Incremental-Pull
  • sip instalment
  • sip instalment Payment Failure
  • lumpsum payment retry (both payment failure and retries)
  • redemption failure

Hi @anuj-ondc @sourabhsaini11 I have added Other flows also. Please verify it.

@anuj-ondc
Copy link

@Abhishekkumar86454, please fix below findings:
common issues:

  • created_at date is different from on_init call.

Sip creation new folio with kyc:

  • on_update:
    • IP_ADDRESS is missing.

sip instalment:
on_status 1:
- created_at date is not matching with previous on_confirm call

sip instalment Payment Failure:

  • on_update:
    • created_at date is not matching with previous on_confirm call
    • Payment state shoube ne FAILED not PAID.

lumpsum payment retry:

  • on_update_new_paymentLink:
    • message_id is not matching with previous action call.

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

Successfully merging this pull request may close these issues.

5 participants