Skip to content

Commit 15ef35d

Browse files
TC-MOclaude
andauthored
fix: update SDK JS URLs after docs restructure (apify-sdk-js#542, #562) (#2280)
The SDK JS docs reorganization renamed paths from guides/apify-platform to numbered directories. Update Nginx rewrites and CI test URLs to match. <!-- CURSOR_SUMMARY --> --- > [!NOTE] > **Low Risk** > Low risk: only adjusts Nginx rewrite targets and CI header-check URLs; impact is limited to routing/redirect behavior for SDK JS docs paths. > > **Overview** > Updates SDK JS documentation routing after the docs restructure. > > `nginx.conf` rewrites for `/sdk/js/docs` and legacy guide URLs now redirect to the new `overview` pages (including `next`), replacing redirects that previously pointed at `guides/apify-platform`. The GitHub Actions Nginx header assertions are updated to hit the new JS SDK docs path (`/sdk/js/docs/introduction/quick-start`) instead of the old guide URL. > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit 1585d6a. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup> <!-- /CURSOR_SUMMARY --> Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
1 parent c0dd962 commit 15ef35d

2 files changed

Lines changed: 9 additions & 7 deletions

File tree

.github/workflows/test.yaml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -89,9 +89,9 @@ jobs:
8989
assert_header "http://localhost:8080/llms-full.txt" "Content-Type" "text/markdown"
9090
9191
echo "🧪 Checking Nginx responses... (apify-sdk-js)"
92-
assert_header "http://localhost:8080/sdk/js/docs/guides/apify-platform" "Content-Type" "text/html"
93-
assert_header "http://localhost:8080/sdk/js/docs/guides/apify-platform.md" "Content-Type" "text/markdown"
94-
assert_header "http://localhost:8080/sdk/js/docs/guides/apify-platform" "Content-Type" "text/markdown" -H "Accept: text/markdown"
92+
assert_header "http://localhost:8080/sdk/js/docs/introduction/quick-start" "Content-Type" "text/html"
93+
assert_header "http://localhost:8080/sdk/js/docs/introduction/quick-start.md" "Content-Type" "text/markdown"
94+
assert_header "http://localhost:8080/sdk/js/docs/introduction/quick-start" "Content-Type" "text/markdown" -H "Accept: text/markdown"
9595
assert_header "http://localhost:8080/sdk/js/llms.txt" "Content-Type" "text/markdown"
9696
assert_header "http://localhost:8080/sdk/js/llms-full.txt" "Content-Type" "text/markdown"
9797

nginx.conf

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -443,10 +443,12 @@ server {
443443
rewrite ^/sdk/js/api(.*)$ /sdk/js/reference$1 permanent;
444444
rewrite ^/sdk/js/reference/apify(.*)$ /sdk/js/reference$1 redirect;
445445
rewrite ^/sdk/js/reference/next/apify(.*)$ /sdk/js/reference/next$1 redirect;
446-
rewrite ^/sdk/js/docs$ /sdk/js/docs/guides/apify-platform$1 redirect;
447-
rewrite ^/sdk/js/docs/next$ /sdk/js/docs/next/guides/apify-platform$1 redirect;
448-
rewrite ^/sdk/js/docs/guides/getting-started$ /sdk/js/docs/next/guides/apify-platform$1 redirect;
449-
rewrite ^/sdk/js/docs/next/guides/getting-started$ /sdk/js/docs/next/guides/apify-platform$1 redirect;
446+
rewrite ^/sdk/js/docs$ /sdk/js/docs/overview redirect;
447+
rewrite ^/sdk/js/docs/next$ /sdk/js/docs/next/overview redirect;
448+
rewrite ^/sdk/js/docs/guides/getting-started$ /sdk/js/docs/overview redirect;
449+
rewrite ^/sdk/js/docs/next/guides/getting-started$ /sdk/js/docs/next/overview redirect;
450+
rewrite ^/sdk/js/docs/guides/apify-platform$ /sdk/js/docs/overview redirect;
451+
rewrite ^/sdk/js/docs/next/guides/apify-platform$ /sdk/js/docs/next/overview redirect;
450452

451453
# old sdk version redirects (we keep only latest, and version only major/minor)
452454
rewrite ^/sdk/js/docs/1\.\d+\.\d+(.*)$ /sdk/js/docs/1.3$1 redirect;

0 commit comments

Comments
 (0)