Comments (4)
Update:
I have found why get(EventEmitter)
didn't work in my codebase, I had two eventEmitter instances running in my app.
I found the bug and fixed it Now get(EventEmitter)
works in my codebase ^^
But the problem persists , the listener is still triggered twice
from event-emitter.
it works when you drop the useless NestFactory.createApplicationContext(AppModule)
call. I can't tell for sure if this is expected or a bug regarding sharing the same root module on both repl
and NestFactory.create*
side note: please next time make sure that your repro has everything to reproduce the issue. It lacks a npm-script to run the repl file
from event-emitter.
You're right. 🤦🏻 removing the useless app creation does fix the issue.
I will make sure to include the npm-script next time.
Could you tell me how this mistake doesn't surface? Two app instances are loaded at once? Why does the debug() not complain or show anything regarding this? I wasn't able to see two AppService instances running.
Thank you
from event-emitter.
@adelbke I'm in the same boat as you. I'd have to investigate deeper to understand how that was possible.
I guess you can create a new issue at the main repo to report this as a repl
issue(?) then we will see Kamil's thoughts about it
from event-emitter.
Related Issues (20)
- EventEmitterModule crash with TypeError from reflect-metadata stack when used with Proxy HOT 1
- Mitigating Event Loss Risk During NestJS Application Initialization HOT 2
- Event Emitter Exception Logging HOT 1
- Add the ability to catch errors from event listeners via ExceptionFilters HOT 2
- @OnEvent decorators is not working when applied multiple times on same method HOT 2
- Duplicate events when using useExisting providers HOT 3
- OnEvent decorator does not triggered if list of events provided HOT 4
- EventEmitterModuleOptions maxListeners setting isn't applied HOT 3
- Multiple event types not handled (impossible to) HOT 4
- Exceptions in async event handlers make the app crash HOT 1
- Custom event decorators break with the latest array metadata type requirement HOT 2
- Jest Unit tests only emit events when intializing full AppModule HOT 8
- Provider EventEmitter can't not resolve when using TerserPlugin webpack HOT 3
- Using event emitter in imported npm package cannot resolve emitter HOT 3
- async function constructor name not AsyncFunction HOT 1
- Upgrade to use event emitter 3, or add an option for it HOT 1
- Suppressing errors when updating from 2.0.0 to 2.0.1 HOT 1
- Option to throw error if an event fails to process HOT 3
- Add OnSafeEvent HOT 7
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from event-emitter.