one more fix for pageviews; include pathname (#9609)

microsoft · Jul 20, 2023 · c1beefd · c1beefd
1 parent 282876d
commit c1beefd
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/docfiles/pxtweb/cookieCompliance.ts b/docfiles/pxtweb/cookieCompliance.ts
@@ -201,6 +201,7 @@ namespace pxt {
                 envelope.baseData.properties = {};
             }
             envelope.baseData.properties.pageName = pageName;
+            envelope.baseData.properties.pathName = window.location.pathname;
             // no url scrubbing for webapp (no share url, etc)
         }
 

diff --git a/docfiles/tracking.html b/docfiles/tracking.html
@@ -22,8 +22,9 @@
                         envelope.baseData.properties.pageName = pageName;
                         var scrubbedUrl = scrubUrl(envelope.baseData.uri);
                         envelope.baseData.uri = scrubbedUrl;
+                        var toUrl = new URL(scrubbedUrl);
+                        envelope.baseData.properties.pathName = toUrl ? toUrl.pathname : "";
                         if (envelope.ext && envelope.ext.trace) {
-                            var toUrl = new URL(scrubbedUrl);
                             envelope.ext.trace.name = toUrl ? toUrl.pathname : "";
                         }
                     }