You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I've been debating refactoring this a bit to have a single utils file with methods to call for each action that sends messages (and only have that file actually call sendMessage).
This should help the code that triggers messages read better and provide a clear choke point if we need to change how messages are sent (e.g. if we move some analysis to a worker in the future)
The text was updated successfully, but these errors were encountered:
I've been debating refactoring this a bit to have a single utils file with methods to call for each action that sends messages (and only have that file actually call
sendMessage
).Originally posted by @antross in #2140
This should help the code that triggers messages read better and provide a clear choke point if we need to change how messages are sent (e.g. if we move some analysis to a worker in the future)
The text was updated successfully, but these errors were encountered: