Skip to content

fix: make site base path configurable + remove unused workflow#56

Merged
github-actions[bot] merged 2 commits into
mainfrom
fix/docs-images-base-path
Mar 4, 2026
Merged

fix: make site base path configurable + remove unused workflow#56
github-actions[bot] merged 2 commits into
mainfrom
fix/docs-images-base-path

Conversation

@spboyer

@spboyer spboyer commented Mar 4, 2026

Copy link
Copy Markdown
Member

Problem

The docs site at proud-beach Azure SWA has all 13 images broken on the dashboard-explore page. astro.config.mjs hardcodes base: '/waza' for GitHub Pages, so ZoomImage.astro prepends /waza to all image paths. Azure SWA serves at root, so /waza/images/... returns 404.

Fix

  • site/astro.config.mjs: Read site and base from ASTRO_SITE and ASTRO_BASE env vars, defaulting to GitHub Pages values. Azure SWA builds can set ASTRO_BASE=/.
  • Remove .github/workflows/squad-heartbeat.yml: No longer used.

Verified

  • Default build (GH Pages): images → /waza/images/explore/...
  • ASTRO_BASE=/ build (SWA): images → /images/explore/...

- Make astro.config.mjs read site/base from ASTRO_SITE and ASTRO_BASE
  env vars with GitHub Pages defaults. Azure SWA can now build with
  ASTRO_BASE=/ so images resolve correctly at root.
- Remove unused squad-heartbeat.yml workflow.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings March 4, 2026 21:33
@github-actions github-actions Bot enabled auto-merge (squash) March 4, 2026 21:35

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Makes the Astro/Starlight docs site deployable under different base paths (GitHub Pages vs Azure Static Web Apps), and removes an unused GitHub Actions workflow.

Changes:

  • Read site and base from ASTRO_SITE / ASTRO_BASE env vars with GitHub Pages defaults.
  • Delete .github/workflows/squad-heartbeat.yml.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
site/astro.config.mjs Makes the docs site URL/base configurable via environment variables.
.github/workflows/squad-heartbeat.yml Removes an unused scheduled/triage workflow.

Comment thread site/astro.config.mjs Outdated
Address review feedback: normalize ASTRO_BASE to prevent double-slash
URLs when base is '/'. Also fix Header.astro to strip trailing slash
from BASE_URL before concatenating nav link paths.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@github-actions github-actions Bot merged commit 7d033df into main Mar 4, 2026
5 checks passed
@richardpark-msft richardpark-msft deleted the fix/docs-images-base-path branch March 4, 2026 22:25
spboyer added a commit to spboyer/waza-fk that referenced this pull request Mar 5, 2026
…soft#56)

* fix: make site base path configurable + remove unused workflow

- Make astro.config.mjs read site/base from ASTRO_SITE and ASTRO_BASE
  env vars with GitHub Pages defaults. Azure SWA can now build with
  ASTRO_BASE=/ so images resolve correctly at root.
- Remove unused squad-heartbeat.yml workflow.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix: normalize base path and prevent // in nav links

Address review feedback: normalize ASTRO_BASE to prevent double-slash
URLs when base is '/'. Also fix Header.astro to strip trailing slash
from BASE_URL before concatenating nav link paths.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants