Compare commits
3
Commits
6207e04bf5
...
v2.3.16
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
52b66da16b | ||
|
|
d75a83ccf2 | ||
|
|
5991bd8392 |
+1
-1
@@ -139,7 +139,7 @@
|
||||
"prepare": "husky",
|
||||
"preinstall": "npx only-allow pnpm"
|
||||
},
|
||||
"version": "2.2.13",
|
||||
"version": "2.3.16",
|
||||
"pnpm": {
|
||||
"onlyBuiltDependencies": [
|
||||
"@parcel/watcher",
|
||||
|
||||
+14
-4
@@ -38,11 +38,21 @@ function getExpectedTranslation(
|
||||
sourcePath: string,
|
||||
sourceLocale: string,
|
||||
targetLocale: string,
|
||||
): string {
|
||||
alternates: { hreflang: string; href: string }[],
|
||||
): string | null {
|
||||
const segments = sourcePath.split('/').filter(Boolean);
|
||||
// First segment is locale
|
||||
segments[0] = targetLocale;
|
||||
|
||||
// Blog posts have dynamic slugs. If it's a blog post, trust the alternate tag
|
||||
// if the href is present in the sitemap.
|
||||
// The Smoke Test's primary job is ensuring the alternate links point to valid pages.
|
||||
if (segments[1] === (targetLocale === 'de' ? 'blog' : 'blog') && segments.length > 2) {
|
||||
const altLink = alternates.find((a) => a.hreflang === targetLocale);
|
||||
if (altLink) {
|
||||
return new URL(altLink.href).pathname;
|
||||
}
|
||||
}
|
||||
|
||||
const map = sourceLocale === 'de' ? SLUG_MAP : REVERSE_SLUG_MAP;
|
||||
|
||||
return (
|
||||
@@ -50,7 +60,7 @@ function getExpectedTranslation(
|
||||
segments
|
||||
.map((seg, i) => {
|
||||
if (i === 0) return seg; // locale
|
||||
return map[seg] || seg; // translate or keep (product names like n2x2y stay the same)
|
||||
return map[seg] || seg; // translate or keep
|
||||
})
|
||||
.join('/')
|
||||
);
|
||||
@@ -118,7 +128,7 @@ async function main() {
|
||||
if (alt.hreflang === locale) continue; // Same locale, skip
|
||||
|
||||
// 1. Check slug translation is correct
|
||||
const expectedPath = getExpectedTranslation(path, locale, alt.hreflang);
|
||||
const expectedPath = getExpectedTranslation(path, locale, alt.hreflang, alternates);
|
||||
const actualPath = new URL(alt.href).pathname;
|
||||
|
||||
if (actualPath !== expectedPath) {
|
||||
|
||||
Reference in New Issue
Block a user