typeorm-transactional-cls-hooked icon indicating copy to clipboard operation
typeorm-transactional-cls-hooked copied to clipboard

Typeorm version update

Open minseop6 opened this issue 3 years ago • 3 comments

Hi @odavid From typeorm version 0.3.x, connection manager is deprecated, So we must be used dataSource. Please check the revised pr.

https://github.com/odavid/typeorm-transactional-cls-hooked/issues/107

minseop6 avatar Jul 19 '22 01:07 minseop6

Hi @minseop6,

I apologize for the late response and I thank you for this contribution. At this stage, I sadly came to realization that I cannot maintain this project anymore.

As for the PR itself...

  • The package was designed to be framework independent, so it can be used on top of Typeorm alone. Adding NestJS as dependency violates this design.
  • As far as I remember, Typeorm also changed the way repositories created (object literal instead of classes), so I am not sure how this PR really uses the BaseRepository/patches to the Repository.

I am really sorry that I cannot help more than that at this stage...

odavid avatar Jul 29 '22 06:07 odavid

here's an up-to-date(?) fork: https://github.com/Aliheym/typeorm-transactional

micalevisk avatar Aug 29 '22 22:08 micalevisk

@odavid can you please deprecate this package on npm? Thank you for your work on this, no worries if you don't have the time to maintain it anymore. Looks like the forked package(s) can take over its role.

ckawell-sb avatar Sep 01 '22 18:09 ckawell-sb