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

feat(mojaloop/#2796): duplicate transaction not getting callback for post /bulkTransfers #332

Conversation

mdebarros
Copy link
Member

@mdebarros mdebarros commented Aug 10, 2022

feat(mojaloop/#2796): duplicate transaction not getting callback for post /bulkTransfers - mojaloop/project#2796

  • updated duplicateCheckComparator to support overriding the generatedHash calculation to accommodate Bulk Transfer Prepare/Fulfil use-cases
  • updated unit tests to validate the above change

chore: maintenance dependencies

  • updated deps
  • added import aliases for cleaner requires

…post /bulkTransfers

fix(mojaloop/##2796): duplicate transaction not getting callback for post /bulkTransfers - mojaloop/project#2796
- added 'BULK_PREPARE_DUPLICATE' event enum action type
- updated dependencies
- replace npm-audit-resolver with audit-ci
- updated CI config for audit-ci changes
- added audit-ci.jsonc and updated allowList
- refactored .ncurc.js to .ncurc.yaml
- added header comment to .ncurc.yaml
…post /bulkTransfers

feat(mojaloop/#2796): duplicate transaction not getting callback for post /bulkTransfers - mojaloop/project#2796
- updated duplicateCheckComparator to suppose overriding the generatedHash calculation to accomodate Bulk Transfer Prepare/Fulfil use-cases
- updated unit tests to validate the above change
…-transaction-not-getting-callback-for-post-bulkTransfers

# Conflicts:
#	package-lock.json
#	package.json
@mdebarros mdebarros marked this pull request as ready for review August 10, 2022 16:03
kleyow
kleyow previously approved these changes Aug 10, 2022
elnyry-sam-k
elnyry-sam-k previously approved these changes Aug 10, 2022
@mdebarros mdebarros dismissed stale reviews from elnyry-sam-k and kleyow via bd92e76 August 11, 2022 11:39
@mdebarros mdebarros merged commit 5427a53 into master Aug 11, 2022
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

Successfully merging this pull request may close these issues.

3 participants