6e679f952f
Up until now we relied on implicit loading of middlewares and reducers, through having imports in each feature's index.js. This leads to many complex import cycles which result in (sometimes) hard to fix bugs in addition to (often) breaking mobile because a web-only feature gets imported on mobile too, thanks to the implicit loading. This PR changes that to make the process explicit. Both middlewares and reducers are imported in a single place, the app entrypoint. They have been divided into 3 categories: any, web and native, which represent each of the platforms respectively. Ideally no feature should have an index.js exporting actions, action types and components, but that's a larger ordeal, so this is just the first step in getting there. In order to both set example and avoid large cycles the app feature has been refactored to not have an idex.js itself.
95 lines
3.1 KiB
JavaScript
95 lines
3.1 KiB
JavaScript
// @flow
|
|
import { appNavigate } from '../app/actions';
|
|
import {
|
|
CONFERENCE_JOINED,
|
|
KICKED_OUT,
|
|
VIDEO_QUALITY_LEVELS,
|
|
conferenceLeft,
|
|
getCurrentConference,
|
|
setPreferredVideoQuality
|
|
} from '../base/conference';
|
|
import { hideDialog, isDialogOpen } from '../base/dialog';
|
|
import { setActiveModalId } from '../base/modal';
|
|
import { pinParticipant } from '../base/participants';
|
|
import { MiddlewareRegistry, StateListenerRegistry } from '../base/redux';
|
|
import { SET_REDUCED_UI } from '../base/responsive-ui';
|
|
import { FeedbackDialog } from '../feedback';
|
|
import { setFilmstripEnabled } from '../filmstrip';
|
|
import { setToolboxEnabled } from '../toolbox';
|
|
|
|
import { notifyKickedOut } from './actions';
|
|
|
|
MiddlewareRegistry.register(store => next => action => {
|
|
const result = next(action);
|
|
|
|
switch (action.type) {
|
|
case CONFERENCE_JOINED:
|
|
case SET_REDUCED_UI: {
|
|
const { dispatch, getState } = store;
|
|
const state = getState();
|
|
const { reducedUI } = state['features/base/responsive-ui'];
|
|
|
|
dispatch(setToolboxEnabled(!reducedUI));
|
|
dispatch(setFilmstripEnabled(!reducedUI));
|
|
|
|
dispatch(
|
|
setPreferredVideoQuality(
|
|
reducedUI
|
|
? VIDEO_QUALITY_LEVELS.LOW
|
|
: VIDEO_QUALITY_LEVELS.HIGH));
|
|
|
|
break;
|
|
}
|
|
|
|
case KICKED_OUT: {
|
|
const { dispatch } = store;
|
|
|
|
dispatch(notifyKickedOut(
|
|
action.participant,
|
|
() => {
|
|
dispatch(conferenceLeft(action.conference));
|
|
dispatch(appNavigate(undefined));
|
|
}
|
|
));
|
|
|
|
break;
|
|
}
|
|
}
|
|
|
|
return result;
|
|
});
|
|
|
|
/**
|
|
* Set up state change listener to perform maintenance tasks when the conference
|
|
* is left or failed, close all dialogs and unpin any pinned participants.
|
|
*/
|
|
StateListenerRegistry.register(
|
|
state => getCurrentConference(state),
|
|
(conference, { dispatch, getState }, prevConference) => {
|
|
const { authRequired, membersOnly, passwordRequired }
|
|
= getState()['features/base/conference'];
|
|
|
|
if (conference !== prevConference) {
|
|
// Unpin participant, in order to avoid the local participant
|
|
// remaining pinned, since it's not destroyed across runs.
|
|
dispatch(pinParticipant(null));
|
|
|
|
// XXX I wonder if there is a better way to do this. At this stage
|
|
// we do know what dialogs we want to keep but the list of those
|
|
// we want to hide is a lot longer. Thus we take a bit of a shortcut
|
|
// and explicitly check.
|
|
if (typeof authRequired === 'undefined'
|
|
&& typeof passwordRequired === 'undefined'
|
|
&& typeof membersOnly === 'undefined'
|
|
&& !isDialogOpen(getState(), FeedbackDialog)) {
|
|
// Conference changed, left or failed... and there is no
|
|
// pending authentication, nor feedback request, so close any
|
|
// dialog we might have open.
|
|
dispatch(hideDialog());
|
|
}
|
|
|
|
// We want to close all modals.
|
|
dispatch(setActiveModalId());
|
|
}
|
|
});
|