threads.js icon indicating copy to clipboard operation
threads.js copied to clipboard

fix: master thread message handler only logs 1 debug message per jobUID

Open clintonwoo opened this issue 1 year ago • 2 comments

Here's a pull request to fix an issue with the worker message event handler.. it's logging 1 debug message per worker event handler.

So if a user has 10 observable events from the worker that the master subscribes to it will log 10 times the same message.

Doing the debug log after filtering the event UID fixes the issue.

Would appreciate a merge and new version if you could be so kind! Thank you!

clintonwoo avatar Apr 09 '24 21:04 clintonwoo

Good point!

andywer avatar Apr 27 '24 22:04 andywer

Hm seems like the CI checks are failing for an unrelated reason

clintonwoo avatar May 05 '24 02:05 clintonwoo