rewiko / crud Goto Github PK

It misses custom operators documentation. Where could we add it ?

First of all thanks for this fork! It fixes a number of crucial issues

#18 fixes some mysql issues, but introduces an issue for requests with manyToOne and manyToMany joins. Because of that we need to stick with 5.1.1 version

Can it be reverted? Or maybe some conditional logic can be added to decide wether skip or offset should be used?

cc: @Darkein

What is your opinion about this new version?

It added a lot of features that can be used, but it also breaks a lot of things.

Do you have plans to work on adding support for this new version?

Due to upstream repo inactivity, I had to fork it to release fixes.

• typeorm - fixed cache - do not use cache when updating resources. Upstream PR nestjsx#688 Repo PR #3
• FIX - select boolean type in JoinOption. Upstream PR nestjsx#713 Repo PR #4
• fix(crud-typeorm): include embeded column name in search query. Upstream PR nestjsx#601
• Update README.md nestjsx#611
• fix(crud): typo fix getMayRouteOptions -> getManyRouteOptions nestjsx#606
• fix(crud): using metadata to update entities based on primary key instead of "found". Upstream PR nestjsx#622
• Fix constant when getting swagger extra models, Upstream PR https://github.com/nestjsx/crud/pull/626/files
• fix filter for user in meController, Upstream PR nestjsx#644
• Make updateOneBase dto type partial, Upstream PR nestjsx#679
• Add warning when using disallowed relation, Upstream PR nestjsx#683
• feat(crud-typeorm): ignore entity decorators when saving, Upstream PR nestjsx#692
• fix(crud-typeorm): fix sql injection check looping on the first regex…, Upstream PR nestjsx#694
• fix(crud-typeorm): use random parameter name to prevent invalid parameters, Upstream PR nestjsx#723
• fix(NestJSX-CRUD-TypeOrm): bug Ability to Use Search and Filter Features with Nested Entities, Upstream PR nestjsx#737
• CHG - Provide options for qs url params stringify, Upstream PR nestjsx#749
• Allow all relation to be joined, Upstream PR nestjsx#574
• feat: add custom operators, Upstream PR nestjsx#568
• fix(typeorm): correct quotes for column identifiers when driver is ‘mariadb‘, Upstream PR nestjsx#546
• Fix(crud-response-interceptor): classToPlain call two time, Upstream PR nestjsx#526

Currently this library support nestjs 7 as listed in package.json, although it supports version 8 without any problems.

But, it might be better to create a new major version of this library and keep support for nestjs 7 in version 5.x and add support for nestjsx 8 in version 6.x.

What do you think?

we should have an error message if the custom operator does not start with $

it's not possible to add a custom operator with an empty value, It should be possible

The other crud package is probably abandoned. Will this fork be actively managed?