feathersjs-ecosystem / feathers-authentication-management

Adds sign up verification, forgotten password reset, and other capabilities to local feathers-authentication
https://feathers-a-m.netlify.app/
MIT License
247 stars 98 forks source link

Not able to verify user. Using provided client library #97

Closed syz3r closed 6 years ago

syz3r commented 6 years ago

Steps to reproduce

Expected behavior

Should update user model.

Actual behavior

Giving error "An id must be provided to the 'patch' method"

On deeper inspection I found inside verifySignup function usersIdName and users.id is undefined which should be "_id". Also I couldn't find any property inside service object which gives ID field name.

System configuration

Tell us about the applicable parts of your setup.

Module versions (especially the part that's not working):

NodeJS version: 9.11.1

Operating System: Mac

Browser Version: Chrome

React Native Version:

Module Loader:

syz3r commented 6 years ago

80 #81

it is working after removing these lines from client code

import authManagementService from 'feathers-authentication-management'; app.configure(authManagementService(options)) const authManagement = app.service('authManagement');

syz3r commented 6 years ago

That's awesome! waiting for the release..

greyivy commented 6 years ago

Is there any way to fix this when using feathers-redux?

eddyystop commented 6 years ago

I think syz3r's change was introduced quite a while ago.

@greyivy, I would think you can use feathers-redux to make the same authManagement service calls that the client functions do. If you have any comments please add them to https://github.com/feathers-plus/authentication-local-management/issues/3

Full details on the authenticate-local-managment rewrite are in https://github.com/feathers-plus/authentication-local-management/blob/master/misc/upgrading.md