feathers-sequelize

feathers-sequelize copied to clipboard

### Summary Complete refactor to `typescript`. It's pretty straight forward. It's meant for feathers v4. So it can be released immediately. I only changed a few things: - modernized: use...

fratzinger

Transactions support

19

Readme says that adapter supports transactions. but not found any other mentions even in sources. Is it any guides how to use it?

mogadanez

iLike method with accent.

1

### Expected behavior It should return all names even if passed without accent in the search. ex: joao, it should search all the users that contain these characters along with...

GabrielGoncalvesMendes

feat: expose 'paramsToAdapter' method

1

This is a refactor which exposes a new `paramsToAdapter` function. It's purpose is to transform feathers params to a `sequelize` query and have that available from the outside. It's useful...

fratzinger

Dove Support

7

- [x] Tell us about the problem your pull request is solving. Updates `@feathersjs` dependencies to `^5.0.0-pre.3` to work with Dove - [x] Are there any open issues that are...

forgot

Use consistent $select in all methods, optimize select function

2

The `_find` method uses Sequelize's `attributes` option to select properties. https://github.com/feathersjs-ecosystem/feathers-sequelize/blob/db4b5ff9b89df9b163f9cd5d8c32de8c915ff71c/lib/index.js#L135 But it does not use the `select` function from adapters-commons on its results. All other methods do use this...

DaddyWarbucks

Optimize multi patch when $returning is false.

3

I noticed this while making this comment: https://github.com/feathersjs/feathers/pull/2472#issuecomment-1008349932 I believe we are potentially wasting some calls to `_getOrFind` if the user is using `$returning: false`. Note here: https://github.com/feathersjs-ecosystem/feathers-sequelize/blob/db4b5ff9b89df9b163f9cd5d8c32de8c915ff71c/lib/index.js#L222 that we...

DaddyWarbucks

Use proper $and in patch and remove methods

2

@fratzinger Made an excellent PR recently that ensures the user's `$and` query is not overwritten. See https://github.com/feathersjs-ecosystem/feathers-sequelize/commit/8de01449511b1bc1a301d077b72e4af67b0f95d1 We should similarly update the `_patch` and `_remove` methods to do so. Note...

DaddyWarbucks

fix multi-patch with default pagination

2

See #363. I consider this quite hacky but it would solve my problem with multi-patch for a length of items > 10 for now. I would prefer to get this...

fratzinger

Sorting should detect attributes in service model

2

When sorting a sequelize query using query built attributes, we need to use `sequelize.col('attribute_name')` instead of a string, so sequelize won't prefix with the table name (`"table_name"."attribute_name"`). getOrder util method...

miguelpeixe