Skip to content

Commit b97ebce

Browse files
authored
docs: Update base path for craft.sentry.dev domain (#734)
## Summary The docs site has moved from \`getsentry.github.io/craft\` to \`craft.sentry.dev\`. This caused assets to fail loading as they were still being served from \`/craft/\`. ## Changes - Change default base path from \`/craft\` to \`/\` - Update site URL to \`https://craft.sentry.dev\` PR previews continue to work via \`DOCS_BASE_PATH\` environment variable override in the workflow.
1 parent abd8cd1 commit b97ebce

2 files changed

Lines changed: 4 additions & 3 deletions

File tree

.github/workflows/docs-preview.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ jobs:
3232
working-directory: docs
3333
env:
3434
# Override base path for PR preview
35-
DOCS_BASE_PATH: /craft/pr-preview/pr-${{ github.event.pull_request.number }}
35+
DOCS_BASE_PATH: /pr-preview/pr-${{ github.event.pull_request.number }}
3636
run: |
3737
pnpm install --frozen-lockfile
3838
pnpm build
@@ -74,4 +74,5 @@ jobs:
7474
source-dir: docs/dist/
7575
preview-branch: gh-pages
7676
umbrella-dir: pr-preview
77+
pages-base-url: craft.sentry.dev
7778
action: auto

docs/astro.config.mjs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,10 +2,10 @@ import { defineConfig } from 'astro/config';
22
import starlight from '@astrojs/starlight';
33

44
// Allow base path override via environment variable for PR previews
5-
const base = process.env.DOCS_BASE_PATH || '/craft';
5+
const base = process.env.DOCS_BASE_PATH || '/';
66

77
export default defineConfig({
8-
site: 'https://getsentry.github.io',
8+
site: 'https://craft.sentry.dev',
99
base: base,
1010
integrations: [
1111
starlight({

0 commit comments

Comments
 (0)