Subscribe about vest

[ealush]

That's actually something I've had in mind for a while.

I was working (then abandoned) on a vest devtools extension that relies on that, so if you look up 2 year old branches, you'll find reference to exactly that (built upon the vest bus).

I think I can add that, but it will be a few days, since I am currently at a conference abroad.

If you want to submit a PR, feel free to do so. I'll gladly review it.

What I had in mind is:

Suite.subscribe(cb)

Cb will receive as arguments:

• Event name
• Event payload

[ealush]

Shipped in 5.1.0. Closing.

[ealush]

Thinking of it again, the event payload may not even be needed in this case, and we can decide not to add it for now. It is enough to know about the event, and then you can pull whatever data you need directly from the suite. No need to expose the internals there.

[ealush]

@vonagam, let me know if you want to take it. Otherwise, I'll implement it.

[vonagam]

the event payload may not even be needed in this case

Ye, for my use case it is not needed.

let me know if you want to take it. Otherwise, I'll implement it.

Do it. (I usually contribute when I become impatient from waiting too long for a thing that I need but you are super fast with responses and have the full picture in mind so I consider it a better choice.)

[ealush]

Hey @vonagam, created this PR. #1080
Please take a look at it. If this looks like what you had in mind as well, I'll release it as part of 5.1.0.