Steven Tsang
Steven Tsang
> i cant say i like the naming, the word `multiple` feels weird here, also the whole point of that module is to deal with the middleware, why should we...
> the user still needs to do the `MikroOrmModule.forRoot` calls explicitly, right? sounds like an implementation detail to me, from user point of view, it doesn't do anything else than...
@B4nan, I have added additional tests and a test to show the issue arising; I have also found that this issue also occurs with the original middleware registration within `mikro-orm-core.module`...
@B4nan updated based off your review
> looks like your approach is not working with nest 11 i will give it ago with nestjs 11
> maybe the problem you were trying to solve is no longer valid with v11? that's the first thing I would try to verify Yeah I notice some module changes...
> > maybe the problem you were trying to solve is no longer valid with v11? that's the first thing I would try to verify > > Yeah I notice...
> maybe the problem you were trying to solve is no longer valid with v11? that's the first thing I would try to verify The issue has been resolved https://github.com/nestjs/nest/releases/tag/v11.0.5....
@B4nan I have updated the `peerDependency` for nestjs 11 to go from `^11.0.5` so anyone installing shouldn't be using anything lower
> do i get it right that things without your PR worked fine, and with the changes you did here it broke with v11 before 11.0.5? My changes were mainly...