search

needle-innovision / nestjs-tenancy

Multi-tenancy approach for nestjs - currently supported only for mongodb with mongoose
MIT License
187 stars 58 forks source link

Microservices Support #38

Open moparlakci opened 1 year ago

moparlakci commented 1 year ago

PR for #37

Hi, can you review the changes, and maybe fix the request injection and request type check if you are more familiar with NestJs and Typescript than me :) Tests are passing for TCP clients, other clients like RMQ, Websockets, Graphql should pass when the request injection is done better.. The code is there but couldnt plug it together for all of them..

moparlakci commented 1 year ago

@sandeepsuvit ping

sandeepsuvit commented 1 year ago

Hi @moparlakci thanks for the PR. I see that multiple core methods have been modified in this PR. Hence ill have to review this PR carefully so that the normal workflow isn't affected. Please give me some time to review it since i am currently busy with my other projects. Will update you here after i review it.

moparlakci commented 1 year ago

@sandeepsuvit okay understandable, thanks. Normally all tests are passing.

ezebrotsky commented 1 year ago

Hi @sandeepsuvit, is it possible to review this PR and merge it if it's working well?

Robokishan commented 1 year ago

Any update over this. This PR looks good! can you please merge this ?

jayzernguyenstarack commented 1 year ago

Hi guys just follow up on this. When will this be ready?

Robokishan commented 1 year ago

I just grab the forked repo for this change. everything is working good in production thanks for fork man! @moparlakci

moparlakci commented 1 year ago

Hi Guys @Robokishan @jayzernguyenstarack @ezebrotsky This still needs some improvements, not 100% foolproof. I'll try to make some time for this, this week.

Have a nice day

jayzernguyenstarack commented 1 year ago

Hi Guys @Robokishan @jayzernguyenstarack @ezebrotsky This still needs some improvements, not 100% foolproof. I'll try to make some time for this, this week.

Have a nice day

Thanks mate

manneyugn commented 10 months ago

hello, any update on this, I happened to run into the same use case mentioned in this pull request