There’s nothing wrong with using BindableEvents. However, some people like to make their own event modules because they can customize it and add new features.
I tried making one myself using coroutines, but I wasn’t able to properly implement :Wait() or :DisconnectAll(). I have a post about it here: Issues with my event system