feathersjs / feathers

The API and real-time application framework
https://feathersjs.com
MIT License
15.07k stars 752 forks source link

fix(core): Improve service option usage and method option typings #2902

Closed daffl closed 1 year ago

daffl commented 1 year ago

This pull request tightens up the handling of service options and makes sure the methods option only allows valid service method names

app.use('myservice', new MyService(), {
  methods: ['get', 'finds'] // Will type error if `finds` does not exist
})
netlify[bot] commented 1 year ago

Deploy Preview for feathers-dove canceled.

Name Link
Latest commit d3c8bb5c651a370efe72acaf5efcd3a329ffb1c2
Latest deploy log https://app.netlify.com/sites/feathers-dove/deploys/63898b374ae604000817c908