Lucian Mocanu
Lucian Mocanu
@amiryadid @TanguyPoly I would adjust the logic in the drivers to be the same everywhere - @sgarner made that nice overview above. The drivers marked with 🙅 and 😕 should...
@sgarner @TanguyPoly what if we revert to the last commit where the tests were passing and merge this PR? There are quite a lot of changes, if we need to...
Maybe it can be supported via an external package rather than by TypeORM itself?
Well, so this idea might work in some basic use cases, but adding a new column will always add it last, regardless where you want to have it. You can't...
`0.3.22` should come out soon. Until then, feel free to use the pre-release `0.3.22-dev.7b242e1`. For a list of changes besides Expo 52 support, take a look at https://github.com/typeorm/typeorm/compare/0.3.21...7b242e1.
@xdc0 could you check if https://github.com/typeorm/typeorm/issues/5637#issuecomment-3189859054 solves the original issue for you?
See https://github.com/typeorm/typeorm/pull/10927 for the full fix.
I see quite a lot of potential issues in the current state of the PR. There's a lot of differences between drivers which shouldn't be there - I'm especially concerned...
From the discord chat: > I just figured out that the issue come from the fact of applying a name for an entity in the decorator Adding a name will...
I also tested this build in my repo (>100 seeders) and it works fine. Also, since it relies on TypeORM loading the classes, there's no need to use default exports.