Add comment to explain QSP logic

tjleing · web-flow · commit c1b4b6ff6bfd · 2023-05-30T15:43:05.000-07:00
diff --git a/src/components/Menu/VersionSwitcher/index.tsx b/src/components/Menu/VersionSwitcher/index.tsx
@@ -130,6 +130,8 @@ export function LibVersionSwitcher({
     urlEnd = url.split("/lib")[1];
   }
 
+  // Function to remove query string parameters before checking if href is included in the list of possibilities.
+  // This is so we are only comparing the paths without the query string parameters to avoid false negatives.
   function isHrefIncluded(href: string, paths: string[]) {
     href = href.split("#")[0];
     if (href.endsWith("/")) {

Original file line number	Diff line number	Diff line change
`@@ -130,6 +130,8 @@ export function LibVersionSwitcher({`
`130`	`130`	`urlEnd = url.split("/lib")[1];`
`131`	`131`	`}`
`132`	`132`
	`133`	`+ // Function to remove query string parameters before checking if href is included in the list of possibilities.`
	`134`	`+ // This is so we are only comparing the paths without the query string parameters to avoid false negatives.`
`133`	`135`	`function isHrefIncluded(href: string, paths: string[]) {`
`134`	`136`	`href = href.split("#")[0];`
`135`	`137`	`if (href.endsWith("/")) {`