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/project#2556): implement patch notification for failure scenarios #492

Merged
merged 32 commits into from
Feb 22, 2022

Conversation

mdebarros
Copy link
Member

@mdebarros mdebarros commented Feb 8, 2022

PR re-based from #489 from @lewisdaly.

feat(mojaloop/project/issue: Implement patch notification for failure scenarios (following v1.1 update) - mojaloop/project#2556

  • fixed unit tests

chore: updated dependencies

  • updated dependencies
  • fixed audit issues
  • fixed lint issues

fix(#2697): Central-Ledger Fulfil Handler does not correctly invalidate requests with an incorrect/non-existent FSP-ID in the FSPIOP-Destination header - mojaloop/project#2697

  • fixed/added unit tests
  • improved test coverage

lewisdaly and others added 23 commits November 15, 2021 15:25
…bad behaviour when parsing the config object
…h/ml-api-adapter into vessels-tech/feat/2553-patch-notification

# Conflicts:
#	package-lock.json
#	package.json
- updated dependencies
- fixed audit issues
- re-linked central-services-shared dependency to updated PR branch
@mdebarros mdebarros changed the title feat(mojaloop/project#2556): implement patch notification for failure scenarios (following v1.1 update) feat(mojaloop/project#2556): implement patch notification for failure scenarios Feb 8, 2022
@mdebarros mdebarros marked this pull request as ready for review February 17, 2022 13:46
@mdebarros mdebarros self-assigned this Feb 17, 2022
@elnyry-sam-k
Copy link
Member

Thanks Miguel, looking good.. we should be good to go after the CI steps are good

@mdebarros mdebarros merged commit 026f764 into master Feb 22, 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.

None yet

3 participants