Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Allow to register services that are event emitters #118

Closed
daffl opened this issue Feb 27, 2015 · 1 comment
Closed

Allow to register services that are event emitters #118

daffl opened this issue Feb 27, 2015 · 1 comment
Milestone
1.1.0

Comments

@daffl
Copy link
Member

daffl commented Feb 27, 2015

The event mixin has to ignore remote services (like feathers-websocket-client) that are already event emitters. If a service is already an event emitter and provides the events property (see #111) it should skip it and instead set _serviceEvents to the events provided.
@daffl daffl mentioned this issue Mar 3, 2015
Closed
@daffl daffl added this to the 1.1.0 milestone Mar 11, 2015
daffl added a commit that referenced this issue Jun 22, 2015
@daffl
Adding ability to register services that already are event emitters (#…
0c9110a 
…118)
@daffl daffl mentioned this issue Jun 22, 2015
Merged
@kethan kethan mentioned this issue Mar 20, 2017
Closed
daffl pushed a commit that referenced this issue Aug 21, 2018
@greenkeeper @daffl
Update @feathersjs/commons to the latest version 🚀 (#118)
b113fb6 
* chore(package): update @feathersjs/commons to version 2.0.0

* Update Travis installation script
daffl pushed a commit that referenced this issue Aug 22, 2018
@greenkeeper @daffl
Update @feathersjs/commons to the latest version 🚀 (#118)
1b64103 
* chore(package): update @feathersjs/commons to version 2.0.0

* Update Travis installation script
daffl pushed a commit that referenced this issue Aug 25, 2018
@ekryski @daffl
Merge pull request #119 from feathersjs/fix-118
6b95f7f 
removing const. Closes #118
daffl pushed a commit that referenced this issue Aug 25, 2018
@Albert-Gao @daffl
#118 Tweak the timestamp in mongoose model template (#284)
d53914e
@lock
Copy link

lock bot commented Feb 8, 2019

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue with a link to this issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Feb 8, 2019
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
1.1.0
Development

No branches or pull requests
1 participant
@daffl