Refactor whatsapp event

[su-chang]

Related comments: https://github.com/wechaty/puppet-whatsapp/pull/256#issuecomment-1046786919

[bung87]

I think you only need extends base class when need access base class properties, others class just｀implements MessageEvent` or other XxxEvent

[su-chang]

I think you only need extends base class when need access base class properties, others class just｀implements MessageEvent` or other XxxEvent

But all sub-classes should extend base class due to they need call methods which in this.manager and need get the id by getBotId().

So that is the reason of my design:

• EventEmitter
  • WhatsAppBase
  • WhatsAppManager
  • LoginEventHandler
  • MessageEventHandler
  • GroupEventHandler

Please share your mind to me, thank you. @bung87 @hcfw007

[bung87]

I double checked , they all need emit , this.manager , so that make sense.