LBRYCommunity/lbry-desktop
1
0
Fork 0
Code Issues 596 Pull requests 6 Projects Releases 238 Packages Wiki Activity

Changed navigation analytic location

Browse source
This commit is contained in:
liam 2017-12-11 22:18:25 -08:00
parent ca16c1d375
commit c4cccac632
2 changed files with 4 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
src/renderer/redux/actions/navigation.js
View file

@ -9,6 +9,7 @@ import {
} from "redux/selectors/navigation";
import { doSearch } from "redux/actions/search";
import { toQueryString } from "util/query_params";
import amplitude from "amplitude-js";
export function doNavigate(path, params = {}, options = {}) {
return function(dispatch, getState) {
@ -23,6 +24,8 @@ export function doNavigate(path, params = {}, options = {}) {
const scrollY = options.scrollY;
amplitude.getInstance().logEvent("NAVIGATION", { destination: url });
dispatch({
type: types.HISTORY_NAVIGATE,
data: { url, index: options.index, scrollY },

9
src/renderer/redux/reducers/navigation.js
View file

@ -1,6 +1,5 @@
import * as types from "constants/action_types";
import { parseQueryParams } from "util/query_params";
import amplitude from "amplitude-js";
const currentPath = () => {
const hash = document.location.hash;
@ -71,13 +70,7 @@ reducers[types.WINDOW_SCROLLED] = (state, action) => {
export default function reducer(state = defaultState, action) {
const handler = reducers[action.type];
if (handler) {
let nextState = handler(state, action);
if (nextState.currentPath !== state.currentPath) {
amplitude
.getInstance()
.logEvent("NAVIGATION", { destination: nextState.currentPath });
}
return nextState;
return handler(state, action);
}
return state;
}