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

fix(mojaloop/#3152): initial open settlementWindow is failing on clean install #946

Conversation

mdebarros
Copy link
Member

@mdebarros mdebarros commented Mar 2, 2023

fix(mojaloop/#3152): initial open settlementWindow is failing on clean install - mojaloop/project#3152

  • updated seeds/settlementWindow2Open.js to
    • handle an array being returned from the knex.insert operation
    • to throw unexpected errors to force the migration scripts to hard fail, so we don't have a soft failure in future that requires deep "investigation" to find something that should be easily noticed
  • fixed unit tests, improved descriptions and code-coverage

This issue was introduced in Knex v2.4.0 with the following change:

chore: updated dependencies and fixed unit tests

  • updated dependencies
  • fixed api changes on Glob dependency (reverted due to a license issue with v9.x)
  • Glob dependency added to upgrade exception list due to v9.x introducing a dependency Package "path-scurry@1.6.1" licensed under "BlueOak-1.0.0" which is not permitted by the Mojaloop License Policy

…an install

- updated seeds/settlementWindow2Open.js to handle an array being returned from the knex.insert operation
- fixed dependency import api changes on code-base

- fixed unit tests for previous commit
elnyry-sam-k
elnyry-sam-k previously approved these changes Mar 2, 2023
@mdebarros mdebarros changed the title feat(mojaloop/#3152): initial open settlementWindow is failing on clean install fix(mojaloop/#3152): initial open settlementWindow is failing on clean install Mar 2, 2023
elnyry-sam-k
elnyry-sam-k previously approved these changes Mar 2, 2023
- Glob v9.x introducing a dependency Package "path-scurry@1.6.1" which is licensed under "BlueOak-1.0.0" which is not permitted by the Mojaloop License Policy
elnyry-sam-k
elnyry-sam-k previously approved these changes Mar 2, 2023
@mdebarros mdebarros merged commit 111d393 into master Mar 2, 2023
@mdebarros mdebarros deleted the feat/#3152-initial-open-settlementWindow-is-failing-on-clean-install branch March 3, 2023 07:56
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

2 participants