allow parameters to be send

[shabanovd]

Base on documentation create, get, update, patch and remove service methods have params argument.

async get(id: Id, params: Params) {}
async create(data: any, params: Params) {}
async update(id: NullableId, data: any, params: Params) {}
async patch(id: NullableId, data: any, params: Params) {}
async remove(id: NullableId, params: Params) {}

This PR is adding params argument to all that methods.

[Dahkenangnon]

Hi @shabanovd

Any description about your PR please ?

What the new feature feature do you enable ?

Then I can review and merge.

Thank, for starting to contribute to this.... waiting for your reply

[shabanovd]

Hey @Dahkenangnon, I updated the description. Let me know if you still have questions.

[Dahkenangnon]

Base on documentation create, get, update, patch and remove service methods have params argument.

async get(id: Id, params: Params) {}
async create(data: any, params: Params) {}
async update(id: NullableId, data: any, params: Params) {}
async patch(id: NullableId, data: any, params: Params) {}
async remove(id: NullableId, params: Params) {}

This PR is adding params argument to all that methods.

These params argument are new feature enabled by recent version of the featherjs framework. Even I merged be sure to have the right version on your backend.

[shabanovd]

It's quite old staff, see https://buzzard.docs.feathersjs.com/api/services.html

[Dahkenangnon]

https://buzzard.docs.feathersjs.com/api/services.html

Nice dear

[Dahkenangnon]

It's quite old staff, see https://buzzard.docs.feathersjs.com/api/services.html

Are you interested to become core contributor for this packages ?

[Dahkenangnon]

It's quite old staff, see https://buzzard.docs.feathersjs.com/api/services.html

Are you interested to become core contributor for this packages ?

@shabanovd , I need some help to keep maintaining it. We can continue discussion via email if you want

[shabanovd]

@Dahkenangnon sure, I can help you with it.