feathers-sequelize

feathers-sequelize copied to clipboard

perf: set id, check necessity of $in and $and

19

This PR does 4 things: - perf: `paramsToAdapter`: if `id` is not null, add `id` to the query intelligently. - if `query[this.id] === id` we can skip adding the id...

fratzinger

daffl

daffl

### Summary When we use the find method of a service that extends SequelizeService the return type is not like expected. We got an union of paginated type and not...

yassinedorbozgithub

### Steps to reproduce - Create a feathers service - Implement the find method of a service without adding the field paginated inside the parameters - Check the return type...

yassinedorbozgithub

daffl

daffl

Here is an example to enhance $include for [email protected] ```js export class MyService extends SequelizeService { /** * abstract function for defind associations * @param {*} model current model *...

chunlampang

Hey, Just experienced a breaking change when updating to latest feathers-sequelize so thought it might be helpful if ever other people encounter this bug :) ### Steps to reproduce remove...

chrisbag

Better handle paranoid models

2

We should be cautious of any methods that do get/find AFTER modification. This can cause issues when using `paranoid` models. Sequelize's `paranoid` option automatically adds `WHERE deleatedAt IS NULL` to...

DaddyWarbucks