Can’t pass multiple arguments about vue-socket.io HOT 3 CLOSED

I think the problem comes from here:

In Observer.js, line 21~25

this.Socket.onevent = (packet) => {
    Emitter.emit(packet.data[0], packet.data[1]);  // !!!!

    if(this.store) this.passToStore('SOCKET_'+packet.data[0], packet.data[1])
};

While packet.data contains the event's name and the arguments:

I'm not clear why to use only the first two arguments instead of all of them.

Maybe it should be written in this way:

this.Socket.onevent = (packet) => {
    Emitter.emit(...packet.data);

    if(this.store) this.passToStore('SOCKET_'+packet.data[0], ...packet.data.slice(1))
};

Now it can normally handle multiple arguments.

from vue-socket.io.

+1 for @huhk-sysu , I'm using Laravel Echo in combination with Vue-Socket. And Echo returns three arguments, Event, Channel and data. Now Vue-Socket is kinda useless with this combination because it always returns the channel-name. Example below :

Love to see a new update! 👍

from vue-socket.io.

vuex mutations and actions do not support multi arguments as per the documentation https://vuex.vuejs.org/en/mutations.html You will need to use object payload if you need multiple arguments

The signature of mutations and actions are described there https://vuex.vuejs.org/en/api.html

mutations: {
  increment (state, payload, options) {
    state.count += payload.amount
  }
},
actions: {
  checkout (context, payload, options) {
     // ....
    )
  }
}

All changes for multiple arguments should have never been merged. Build 2.1.1-b is not compatible with store vuex

from vue-socket.io.