Don't trigger onRouteUpdate on hash updates

[brandonmp]

I export an onRouteUpdate from gatsby-browser.js like so:

export const onRouteUpdate = ({ location, action }) => {
    if (
        window.location &&
        typeof window.location.href === 'string' &&
        window.location.href.includes('www.example.com') // make sure we're on live site
    ) {
        window.analytics.page();
    } else {
        console.log('PAGE: ', window.location.href, action, location);
    }
    return;
};

This works fine, but when there are anchor links on a page, it fires every time one of them is clicked.

I tried just firing the page() on action === 'PUSH' route updates, but that blocks the event on all initial page loads.

Any way to remove anchor link events from this tracking?

[sebastienfi]

Hi @brandonmp Maybe check in window.location.href if # character is present ?

[KyleAMathews]

@sebastienfi he's saying that a hash update (what happens when you click on an anchor) shouldn't trigger the onRouteUpdate API. Which seems pretty reasonable to me. Hash updates should only occur within a single page where the API is intended to be triggered only when switching pages.

@brandonmp would love a PR filtering out hash changes within the same page here!

gatsby/packages/gatsby/cache-dir/production-app.js

Line 112 in 8436207

apiRunner(`onRouteUpdate`, { location, action })

Perhaps put the function in a different file and add a few tests?

[KyleAMathews]

Not sure when but we don't fire onRouteUpdate on route changes any more.