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

Feature/410 user friendly message #59

Closed
wants to merge 71 commits into from
Closed

Feature/410 user friendly message #59

wants to merge 71 commits into from

Conversation

bothadeon
Copy link
Member

No description provided.

Shashikant Hirugade and others added 30 commits April 25, 2018 12:19
TODO: 
1. config Kafka config
2. health end-point
3. fix functional-runner.sh
CI, CD setup and refactoring
* Updating correct path (based on repo/project name)

* Removing running five bells ledger tests
* changes for story#276

* changes for story#276 after review

* changes for story#276 after review

* changes for story#276 after review

* changes for story#276 after review-removed unwanted files

* changes for story#276 after review-removed unwanted files

* changes for story#276 after review-changed the kafka topic name
…13)

* Updated the notification handler to accept  fulfilment

* removed space

* Updated the notifications handler

Updated the notifications handler with changes from the review comments

* Merge from mojaloop  dev merge

* Fixed integration tests
* fixes for issues experienced on ml-api adapter fixes for tests

* re-aligning notification logic to '2.1.3. Transfer Handler Consume (Success)' flow. Notification handler expecting action of 'fulfil' where as per SDs it should be 'commit' (#15)

* re-aligning notification logic to '2.1.3. Transfer Handler Consume (Success)' flow. Notification handler expecting action of 'fulfil' where as per SDs it should be 'commit'

* fixed unit tests
shashi165 and others added 25 commits August 29, 2018 22:47
* Fix for #34

* changes for story 323

* Fix for #33

* Fixed unit test coverage

* Fixed test coverage

* Fix for bug #34 (#36)

* Fix for #34

* Fixed test coverage

* fixed the unit tests

* integration test

* integration tests

* integration tests

* fixed the bug with notificatin handler
* Added handling for payee rejections
* Added unit test to cover 100% branches
* Fix for #34

* changes for story 323

* Fix for #33

* Fixed unit test coverage

* Fixed test coverage

* Fix for bug #34 (#36)

* Fix for #34

* Fixed test coverage

* fixed the unit tests

* integration test

* integration tests

* integration tests

* fixed the bug with notificatin handler

* bug #424 fixed
* updates for Date Header

* updates for Date Header

* added more unit tests
… config (#53)

* Fixed the spelling FSPIOP

* separated the caching logic from hapi server

* removed unused helper.js

* removed unused helper.js

* added the unit and coverage tests

* changed the name ENDPOINT_CACHE_URI

* change the name ENDPOINT_CACHE_URI

* removed the used helper.test.js

* refactored the code

* fixed the coverage tests

* code cleanup

* refactored the code as per suggestion by miguel
* fix for broken integration tests

* moved immutable from devDependencies to normal dependencies
* added logging and jsdocs

* cleaned up the package.json, updated the npm versions

* cleaned up the package.json, updated the npm versions
* fix for broken integration tests

* moved immutable from devDependencies to normal dependencies

* updates to fix circleci build failure

* fix for double &&
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.

8 participants