-
Notifications
You must be signed in to change notification settings - Fork 98
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update Library Dependencies and Code for Feathers 5 (Dove) #210
Conversation
* Upgrade feathers to dove * Add typings related for dove migration * First iteration on tests, need more work * Update dependencies to pre.30 * Transform hooks in a generic way for HookContext * Add @sinclair/typebox in dev dependency (need to be removed later) * Set node >= 14 for github CI and package.json as feathers dove supports node >= 14 * Fix tests and implementation * use NullableId for checkUnique * update hashPassword for reusing same context instead of new one (never transmitted) * replace typing for hooks as @fratzinger advice * Update to dove pre.31
✅ Deploy Preview for feathers-a-m ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
Anyone to review this PR? |
As far as I can see I don't see any breaking changes or doubtful refactoring and tests are passing, good job. However, as I am less familiar with latest typescript version I suggest @fratzinger to review it as well. |
I'll merge and do some chore work like updating dependencies right now. I'll release a major version afterwards. Thank you all for your work and sorry for the delay! ❤️ |
Summary
I'm putting the final touches on our upgrade to Feathers v5 Dove, and keep running into inconsistent builds, with probelms traced back to v4.x feathers-common and feathers-adapter-commons libraries getting referenced from this library. This PR brings feathers-authentication-management in-line with the v5 release.
This PR builds on the branch from @mdartic and ensures that the test suite is passing. I am currently testing an internaly published package based on this branch and will update as needed.
Todo
Other Information