feathersjs-ecosystem / feathers-permissions

Simple role and service method permissions for Feathers
MIT License
184 stars 29 forks source link

Dove Support #125

Open forgot opened 2 years ago

forgot commented 2 years ago

Summary

Other Information

This pull request updates the dependencies to ^5.0.0-pre.16. No changes were needed to the actual lib files. All tests now pass with npm test. If it would be better to do this as a full Typescript implementation, I can work on that.

Since the FeathersError properties data and errors are now optional and not used in the test, I removed them from expected result in the tests. I guess another option would be to use a Forbidden error with the expected message? I can make that change if needed.

It would obviously be better for this to remain it's own branch and did not merge with release or master, but there is not currently a Dove branch.