Flyout: do not add UTM parameters to downloads (#347)

These links point to the documentation domain and we don't receive stats from those in our Plausible account, so it's useless to add UTM parameters there. It seems I added them here by mistake.
readthedocs · Jul 17, 2024 · 8c9d66e · 8c9d66e
1 parent 9e229e7
commit 8c9d66e
Show file tree

Hide file tree

Showing 5 changed files with 7 additions and 9 deletions.
diff --git a/dist/readthedocs-addons.js b/dist/readthedocs-addons.js
diff --git a/dist/readthedocs-addons.js.map b/dist/readthedocs-addons.js.map
diff --git a/src/flyout.js b/src/flyout.js
@@ -202,9 +202,7 @@ export class FlyoutElement extends LitElement {
         ${Object.entries(this.config.versions.current.downloads).map(
           ([name, url]) =>
             html`<dd>
-              <a href="${addUtmParameters(url, "flyout")}"
-                >${nameDisplay[name]}</a
-              >
+              <a href="${url}">${nameDisplay[name]}</a>
             </dd>`,
         )}
       </dl>

diff --git a/tests/__snapshots__/flyout.test.snap.js b/tests/__snapshots__/flyout.test.snap.js
diff --git a/tests/flyout.test.html b/tests/flyout.test.html
@@ -175,7 +175,7 @@
       <dl class="downloads">
         <dt>Downloads</dt>
         <dd>
-          <a href="https://project.readthedocs.io/_/downloads/en/latest/pdf/?utm_source=project&amp;utm_content=flyout">PDF</a>
+          <a href="https://project.readthedocs.io/_/downloads/en/latest/pdf/">PDF</a>
         </dd>
       </dl>
       `);