BREAKING CHANGE: first arg of room-leave event licener changed from Contact to Contact[]

[lijiarui]

room-leave has update according: #250 #370
It structures as follows:

.on('room-leave', function(this, room, leaver, remover) {
  log.info('Bot', 'EVENT: room-leave - Room %s lost member %s by %s'
                , room.topic()
                , leaverList.map(c => c.name()).join(',')
                , remover.name()
              )
})

But in src/wechaty.ts, it says:

  public on(event: 'room-leave' , listener: (this: Wechaty, room: Room, leaverList: Contact[]) => void): this

In src/room.ts, it says:

public on(event: 'leave', listener: (this: Room, leaver: Contact) => void): this

This is old version of room-leave

[huan]

It should be synced when we merge your PR #370

Could you please to submit a PR to fix this? Thanks.

[binsee]

Related Information

in src/puppt-web/firer.ts:

  /**
   * FIXME: leaver maybe is a list
   * @lijiarui: I have checked, leaver will never be a list. If the bot remove 2 leavers at the same time, it will be 2 sys message, instead of 1 sys message contains 2 leavers.
   */
  this.emit('room-leave', room, leaverContact, removerContact)
  room.emit('leave'           , leaverContact, removerContact)

[huan]

To make the API be consist, I'd like to suggest we always use contactList s parameters.