Merge pull request #10582 from davidkelly/dk/4996

remove extra activity logger call
This commit is contained in:
David Kelly 2017-05-31 15:18:17 -07:00 committed by GitHub
commit c29091f0b1

View file

@ -613,7 +613,6 @@
error = "All participants must be logged in to connect."; error = "All participants must be logged in to connect.";
} }
result = error ? {status: 'error', connection: error} : response; result = error ? {status: 'error', connection: error} : response;
UserActivityLogger.makeUserConnection(connectingId, false, error || response);
connectionRequestCompleted(); connectionRequestCompleted();
} else { } else {
result = response; result = response;
@ -668,8 +667,8 @@
// to be sure the hand is still close enough. If not, we terminate // to be sure the hand is still close enough. If not, we terminate
// the interval, go back to the waiting state. If we make it // the interval, go back to the waiting state. If we make it
// the entire CONNECTING_TIME, we make the connection. We pass in // the entire CONNECTING_TIME, we make the connection. We pass in
// whether or not the connecting id is actually logged in, as now we // whether or not the connecting id is actually logged in, as now we
// will allow to start the connection process but have it stop with a // will allow to start the connection process but have it stop with a
// fail message before trying to call the backend if the other guy isn't // fail message before trying to call the backend if the other guy isn't
// logged in. // logged in.
function startConnecting(id, jointIndex, isLoggedIn) { function startConnecting(id, jointIndex, isLoggedIn) {