[core] Add dev-time warning for events sent before actor is started #4794
Conversation
|
|
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. |
| if (this._processingStatus === ProcessingStatus.NotStarted) { | ||
| console.warn( | ||
| `Event "${event.type}" sent to actor "${this.id}" before it started. Remember to call \`actor.start()\` before sending events.` | ||
| ); | ||
| } |
There was a problem hiding this comment.
It's not a good warning when it's applied like this. Sending events like this is totally legal - I'm pretty sure that we do this in our tests and that the CI might quickly fail because of this.
But it's not only about our test suite. Sending from a child effect to the parent would start warning (it's not desired) since it happens before the parent's effect runs (so before it has its chance to start the target actor).
No description provided.