Skip to content
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

Bug/fixing example #22

Merged
merged 2 commits into from Jun 6, 2017
Merged

Conversation

cpt-chewieeee
Copy link

This PR fix a small bug in the example.

I was trying to run the example to get a better understanding of the flow. But couldn't run it.

In example/src/services/user/hooks/index.js: const verifyHooks = require('../../../hooks').hooks is undefined.

I change the code to require('../../../hooks').verifyResetHooks exactly like example/src/services/message/hooks/index.js

@coveralls
Copy link

Coverage Status

Coverage remained the same at 97.264% when pulling c8c6c34 on cpt-chewieeee:bug/fixing-example into d1f044a on eddyystop:master.

@eddyystop eddyystop merged commit d519fb9 into eddyystop:master Jun 6, 2017
@eddyystop
Copy link
Owner

This repo has been deprecated, as the README mentions, in favor of feathers-authentication-management.

| Deprecated: feathers-service-verify-reset has been moved into Feathers core as
| feathers-|authentication-management for the Auk release.
| feathers-authentication-management has important new capabilities. Its API has been rationalized and
| deprecated features have been removed. Documentation is available at Feathers docs under
| Authentication, Password Management, Local. (Also available here for now.)

Please use that instead.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants