Skip to content

refactor: rename doc components to improve URL SEO#761

Merged
flavio merged 3 commits into
mainfrom
seo-optimizations
May 22, 2026
Merged

refactor: rename doc components to improve URL SEO#761
flavio merged 3 commits into
mainfrom
seo-optimizations

Conversation

@flavio
Copy link
Copy Markdown
Member

@flavio flavio commented May 22, 2026

Rename source folders and Antora component names for clearer URLs:

  • docs/admc/ + name: admc → docs/admission-controller/ + name: admission-controller
  • docs/sboms/ + name: sboms → docs/sbom-scanner/ + name: sbom-scanner

Other changes:

  • fix: build PR previews and CI tests from local checkout. Without this the netlify preview would fail (in this case, due to renames), or show stale contents (since it always pulled from main, not from the PR branch)
  • fix: use relative path for logo home link. When clicking on the logo on the top left side of the screen, go to the root of the deployed website instead of https://docs.kubewarden.io

Rename source folders and Antora component names for clearer URLs:
- docs/admc/ + name: admc → docs/admission-controller/ + name: admission-controller
- docs/sboms/ + name: sboms → docs/sbom-scanner/ + name: sbom-scanner

Update playbook start_paths and cross-component xrefs accordingly.

Signed-off-by: Flavio Castelli <fcastelli@suse.com>
Assisted-by: Claude Sonnet 4.6
@netlify
Copy link
Copy Markdown

netlify Bot commented May 22, 2026

Deploy Preview for docs-kubewarden-io ready!

Name Link
🔨 Latest commit 15e1b8f
🔍 Latest deploy log https://app.netlify.com/projects/docs-kubewarden-io/deploys/6a103c2595bb2c0008416b5f
😎 Deploy Preview https://deploy-preview-761--docs-kubewarden-io.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@flavio flavio self-assigned this May 22, 2026
The Netlify deploy preview and the GHA test workflow were both fetching
content from the remote main branch, causing PR builds to ignore the
branch's own changes.
- Switch test.yml to `make community-local` so CI builds from the
  checked-out PR branch
- Switch kw-remote-community-netlify-playbook.yml to use `url: ./` and
  `branches: HEAD` so Netlify deploy previews reflect PR content

The production deploy (deploy.yml + kw-remote-community-playbook.yml)
is unchanged and continues to fetch from remote main after merge.

Signed-off-by: Flavio Castelli <fcastelli@suse.com>
Assisted-by: Claude Sonnet 4.6
@jhkrug
Copy link
Copy Markdown
Contributor

jhkrug commented May 22, 2026

Sure, we can do this. The playbooks will need adjusting as well. Also, maybe move docs/kw to docs/kubewarden. If you like, assign to me and I'll do all that and make sure it builds ok.

Thanks for the change to the netlify build. Yes, it should do a community-local build and use HEAD. The playbook for community-remote for gh-pages will need adjusting to the new paths. And in the community-local playbook (for building on your own workstation). Maybe I can reduce the number of playbooks back down to 2.

@viccuad
Copy link
Copy Markdown
Member

viccuad commented May 22, 2026

This sounds like a good change, +1 (can't formally approve, github chokes when going to the files tab for approving...)

Maybe I can reduce the number of playbooks back down to 2.

That would also be useful.

@flavio
Copy link
Copy Markdown
Member Author

flavio commented May 22, 2026

Thanks for the change to the netlify build. Yes, it should do a community-local build and use HEAD. The playbook for community-remote for gh-pages will need adjusting to the new paths. And in the community-local playbook (for building on your own workstation). Maybe I can reduce the number of playbooks back down to 2.

Sorry for not mentioning that into the PR body, I just fired the PR and ran to have lunch 😄

I think it would be good to go down to one playbook, maybe with another PR?

The logo in the top-left navbar was hardcoded to https://docs.kubewarden.io/,
causing it to redirect away from the current build when using local or
PR preview builds.

Replace the hardcoded URL with `{{{siteRootPath}}}`, which Antora resolves
to a relative path to the site root at build time.

Signed-off-by: Flavio Castelli <fcastelli@suse.com>
Assisted-by: Claude Sonnet 4.6
@flavio
Copy link
Copy Markdown
Member Author

flavio commented May 22, 2026

I've pushed one final commit and updated the PR body to mention all the changes. If you're fine with that I would merge this PR and then reduce the number of netlify playbooks with another PR

@jhkrug
Copy link
Copy Markdown
Contributor

jhkrug commented May 22, 2026

Sure. #756 from @viccuad will also need adjusting.

@flavio flavio merged commit 63dec54 into main May 22, 2026
6 checks passed
@github-project-automation github-project-automation Bot moved this from Pending review to Done in Kubewarden Admission Controller May 22, 2026
@flavio flavio deleted the seo-optimizations branch May 22, 2026 12:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

3 participants