hackerAPI icon indicating copy to clipboard operation
hackerAPI copied to clipboard

build: Upgrading dependencies for compatibility with Node 20 and general maintenance

Open msohaill opened this issue 1 year ago • 0 comments

List of changes:

  • Upgrading all dependencies to latest stable/compatible versions.
  • No major API adjustments required apart from mongoose MongoId being instantiated with new and mongoose no longer accepting callbacks. We used to use these queryCallbackFactory and updateCallbackFactory functions that actually ran every single query twice! I've migrated and fixed this behaviour so (technically) all queries should be twice as fast now 😂
  • Winston logger also needed some API updates, but nothing severe.
  • Database service no longer needs to emit db connected event as Mocha has updated their runners and Mongoose buffers queries.
  • Tests are fixed according to #925

Type of change

Please delete options that are not relevant.

  • [x] Breaking change (fix or feature that would cause existing functionality to not work as expected)

How has this been tested?

  • npm run test is successful!
  • Ran through most major workflows using the dashboard and all are successful/exhibit the same behaviour as prior.

Test Configuration:

Firmware version: Hardware: Toolchain: SDK:

Questions for code reviewers?

None come to mind.

Checklist:

  • [ ] My code follows the style guidelines of this project
  • [x] I have performed a self-review of my own code
  • [ ] I have commented my code, particularly in hard-to-understand areas
  • [ ] My changes generate no new warnings
  • [ ] Listed change(s) in the Changelog
  • [ ] I have added tests that prove my fix is effective or that my feature works
  • [ ] New and existing unit tests pass locally with my changes
  • [x] I have made corresponding changes to the documentation
  • [ ] Any dependent changes have been merged and published in downstream modules

msohaill avatar Jan 15 '24 00:01 msohaill