Skip to content

Commit

Permalink
Make sure web also dispatches CONFERENCE_WILL_JOIN.
Browse files Browse the repository at this point in the history
  • Loading branch information
damencho committed Nov 21, 2017
1 parent d7dffba commit b91d588
Show file tree
Hide file tree
Showing 2 changed files with 21 additions and 4 deletions.
2 changes: 2 additions & 0 deletions conference.js
Expand Up @@ -25,6 +25,7 @@ import {
conferenceFailed,
conferenceJoined,
conferenceLeft,
conferenceWillJoin,
dataChannelOpened,
EMAIL_COMMAND,
lockStateChanged,
Expand Down Expand Up @@ -1249,6 +1250,7 @@ export default {
= connection.initJitsiConference(
APP.conference.roomName,
this._getConferenceOptions());
APP.store.dispatch(conferenceWillJoin(room));
this._setLocalAudioVideoStreams(localTracks);
this._room = room; // FIXME do not use this

Expand Down
23 changes: 19 additions & 4 deletions react/features/base/conference/actions.js
Expand Up @@ -229,10 +229,25 @@ function _conferenceWillJoin(conference: Object) {
_addLocalTracksToConference(conference, localTracks);
}

dispatch({
type: CONFERENCE_WILL_JOIN,
conference
});
dispatch(conferenceWillJoin(conference));
};
}

/**
* Signals the intention of the application to have the local participant
* join the specified conference.
*
* @param {JitsiConference} conference - The {@code JitsiConference} instance
* the local participant will (try to) join.
* @returns {{
* type: CONFERENCE_WILL_JOIN,
* conference: JitsiConference
* }}
*/
export function conferenceWillJoin(conference: Object) {
return {
type: CONFERENCE_WILL_JOIN,
conference
};
}

Expand Down

0 comments on commit b91d588

Please sign in to comment.