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

Added pause of emitter error until a listener for manage this was setted #85

Closed
wants to merge 1 commit into from

Conversation

minuyoutility
Copy link

In my project i used this solution, instead of emitting immediately the errors, i pause this until an external listener was setted, so this error can be managed from the caller, and if the resolver for manage this wasn't called this not cause a server crash.

@jaydenseric
Copy link
Owner

Thanks for your contribution! @mike-marcacci and I discussed it in Slack, and decided that there would be edge-cases to this approach that should be solved by #81.

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.

2 participants