Skip to content

fix: docs build#1916

Merged
SkyeYoung merged 5 commits intomasterfrom
young/chore/tmp-disable-dashboard-next-doc
May 17, 2025
Merged

fix: docs build#1916
SkyeYoung merged 5 commits intomasterfrom
young/chore/tmp-disable-dashboard-next-doc

Conversation

@SkyeYoung
Copy link
Member

@SkyeYoung SkyeYoung commented May 16, 2025

Fixes: #[Add issue number here]

Changes:

Screenshots of the change:

Fix https://github.com/apache/apisix-website/actions/runs/15066615081/job/42352784489. Performed a complete reset locally, installed dependencies, and rebuilt the site to test again.

Also add a workflow_dispatch to help us manually trigger a build to check for issues.

@SkyeYoung SkyeYoung requested a review from Copilot May 17, 2025 09:21
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR aims to fix the documentation build process and add manual triggering capability through GitHub Actions. Key changes include creating an empty sidebars.json for the APISIX Dashboard docs and adding a workflow_dispatch trigger to the deploy workflow.

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
doc/docs/apisix-dashboard/sidebars.json Adds a new JSON configuration file for dashboard documentation.
.github/workflows/deploy.yml Introduces manual triggering via workflow_dispatch with branch input.
Comments suppressed due to low confidence (1)

.github/workflows/deploy.yml:18

  • The workflow default branch is set to 'master'. If your repository default is 'main' or follows a different naming convention, consider updating this value to avoid confusion.
default: 'master'

@SkyeYoung SkyeYoung merged commit 2cd2cf8 into master May 17, 2025
3 checks passed
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.

4 participants