Merge pull request #140 from takker99/fix-project-links

takker99 · web-flow · commit 3bf588d6a164 · 2023-09-04T10:49:56.000+09:00
🐛 Faild to update pages which include `[/:project]` or `[/:project/]`
diff --git a/browser/websocket/makeChanges.ts b/browser/websocket/makeChanges.ts
@@ -107,6 +107,8 @@ const findLinksAndImage = (
           }
           case "root": {
             const link = cutId(node.href);
+            // ignore `/project` or `/project/`
+            if (/^\/[\w\d-]+\/?$/.test(link)) return;
             if (projectLinksLc.has(toTitleLc(link))) return;
             projectLinksLc.add(toTitleLc(link));
             projectLinks.push(link);

Original file line number	Diff line number	Diff line change
`@@ -107,6 +107,8 @@ const findLinksAndImage = (`
`107`	`107`	`}`
`108`	`108`	`case "root": {`
`109`	`109`	`const link = cutId(node.href);`
	`110`	+ // ignore `/project` or `/project/`
	`111`	`+ if (/^\/[\w\d-]+\/?$/.test(link)) return;`
`110`	`112`	`if (projectLinksLc.has(toTitleLc(link))) return;`
`111`	`113`	`projectLinksLc.add(toTitleLc(link));`
`112`	`114`	`projectLinks.push(link);`