From b5fd15ce68e518eb8f0fd3e308ebf87800b6d7f3 Mon Sep 17 00:00:00 2001
From: Noah Berman <noahberm@gmail.com>
Date: Mon, 10 Jun 2019 12:05:10 -0400
Subject: [PATCH 1/3] Added framework for URL-change condition for virtualPage
 call

---
 lib/index.js | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/lib/index.js b/lib/index.js
index 6b2f59e..e40ea67 100644
--- a/lib/index.js
+++ b/lib/index.js
@@ -8,6 +8,9 @@ var defaults = require('@ndhoule/defaults');
 var integration = require('@segment/analytics.js-integration');
 var onBody = require('on-body');
 
+/** TO-DO: connect urlChanged to Segment URL-change event **/
+var urlChanged = true;
+
 /**
  * Expose `Chartbeat` integration.
  */
@@ -20,6 +23,7 @@ var Chartbeat = module.exports = integration('Chartbeat')
   .option('uid', null)
   .option('video', false)
   .option('sendNameAndCategoryAsTitle', false)
+  .option('trackGalleryClicksAsPageviews', false)
   .option('subscriberEngagementKeys', [])
 
   .tag('<script src="//static.chartbeat.com/js/{{ script }}">');
@@ -58,7 +62,7 @@ Chartbeat.prototype.page = function(page) {
     this._ready = false;  // switch ready to false so that no pages after the first one can fire until _initialize has loaded chartbeat script
     this.pageCalledYet = true;
     this._initialize();
-  } else {
+  } else if (this.options.trackGalleryClicksAsPageviews || urlChanged) {
     var props = page.properties();
     window.pSUPERFLY.virtualPage(props.path);
   }

From 20f46a8b15aaf710a40c9a87f36f57e2881e83cf Mon Sep 17 00:00:00 2001
From: Noah Berman <noahberm@gmail.com>
Date: Tue, 16 Jul 2019 16:33:06 -0400
Subject: [PATCH 2/3] Added new logic for change in URL for virtualPage call

---
 lib/index.js | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/lib/index.js b/lib/index.js
index e40ea67..601d0df 100644
--- a/lib/index.js
+++ b/lib/index.js
@@ -8,9 +8,6 @@ var defaults = require('@ndhoule/defaults');
 var integration = require('@segment/analytics.js-integration');
 var onBody = require('on-body');
 
-/** TO-DO: connect urlChanged to Segment URL-change event **/
-var urlChanged = true;
-
 /**
  * Expose `Chartbeat` integration.
  */
@@ -41,6 +38,7 @@ Chartbeat.prototype.loaded = function() {
 
 Chartbeat.prototype.initialize = function() {
   this.pageCalledYet = false;
+  this.currentURL = window.location.href;
   this._ready = true; // temporarily switch ready to true so that a single page call can fire
 };
 
@@ -55,6 +53,7 @@ Chartbeat.prototype.initialize = function() {
 
 Chartbeat.prototype.page = function(page) {
   this.updateConfig(page);
+  var urlChanged = this.currentURL !== window.location.href;
 
   // since chartbeat automatically calls a page when it loads, don't load chartbeat script until
   // first Segment page call comes in and configures global config vars using its props

From 60a4a51f2b447f161e87fa5fe9dd6c517a902da6 Mon Sep 17 00:00:00 2001
From: Noah Berman <noahberm@gmail.com>
Date: Fri, 23 Aug 2019 15:39:21 -0400
Subject: [PATCH 3/3] used more specific pathname instead of href for url
 changes

---
 lib/index.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/index.js b/lib/index.js
index 601d0df..306e243 100644
--- a/lib/index.js
+++ b/lib/index.js
@@ -53,7 +53,7 @@ Chartbeat.prototype.initialize = function() {
 
 Chartbeat.prototype.page = function(page) {
   this.updateConfig(page);
-  var urlChanged = this.currentURL !== window.location.href;
+  var urlChanged = this.currentURL !== window.location.pathname;
 
   // since chartbeat automatically calls a page when it loads, don't load chartbeat script until
   // first Segment page call comes in and configures global config vars using its props