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

Adds a subscribeAll() method to the client to handle any events #112

Merged
merged 1 commit into from
Jun 28, 2016

Conversation

AndrewRayCode
Copy link
Contributor

Usage:

client.subscribeAll(function(message) {
    console.log(message.action);
});

Only accepts one handler and will overwrite any previously existing
handlers. Addresses #111

Usage:

    client.subscribeAll(function(message) {
        console.log(message.action);
    });

Only accepts one handler and will overwrite any previously existing
handlers. Addresses webpack-contrib#111
@coveralls
Copy link

coveralls commented Jun 28, 2016

Coverage Status

Coverage increased (+2.1%) to 76.056% when pulling 09ba503 on DelvarWorld:subscribe-all into 46cbc22 on glenjamin:master.

@glenjamin glenjamin merged commit b6e41b2 into webpack-contrib:master Jun 28, 2016
@AndrewRayCode
Copy link
Contributor Author

Sweet! Do you have any thoughts on the next version release? I'd like to use this in a little side project

@glenjamin
Copy link
Collaborator

Will likely cut a release in the morning UK time

@AndrewRayCode
Copy link
Contributor Author

@glenjamin
Copy link
Collaborator

Released as 2.12.0 - also answered that question on SO.

@AndrewRayCode
Copy link
Contributor Author

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.

3 participants