Skip to content

fix(nav): hide shared-content installation section from sidebar#1951

Open
Reshrahim wants to merge 7 commits into
v0.59from
reshrahim/hide-installation-shared-content
Open

fix(nav): hide shared-content installation section from sidebar#1951
Reshrahim wants to merge 7 commits into
v0.59from
reshrahim/hide-installation-shared-content

Conversation

@Reshrahim

Copy link
Copy Markdown
Contributor

Description

The shared-content/ directory is mounted into Hugo's content (see config.toml), which caused the include-only snippets in shared-content/installation/ to be rendered as pages and surfaced as an "Installations" section in the sidebar navigation.

Change

Added shared-content/installation/_index.md with _build: { list: never, render: never } plus a cascade so the section and all its descendants are excluded from rendering and the nav.

Because the read shortcode reads files directly from disk (readFile), all existing {{< read >}} includes continue to work.

Verification

  • Ran hugo server locally: the "Installations" nav item no longer appears.
  • Confirmed pages using {{< read >}} from shared-content/installation/ (e.g. Install rad CLI) still render their included content.

rad-ci-bot and others added 2 commits May 26, 2026 19:46
The shared-content directory is mounted into Hugo's content, causing the
include-only installation snippets to render as an 'Installations' section
in the sidebar navigation. Add an _index.md with _build render/list never
plus cascade so the section and its children are excluded from rendering
and navigation. The read shortcode reads files directly from disk, so all
existing includes continue to work.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: e412b758-7998-4133-ab00-e904a086f732
@Reshrahim Reshrahim requested review from a team as code owners July 10, 2026 16:09
@github-actions

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@github-actions

github-actions Bot commented Jul 10, 2026

Copy link
Copy Markdown

Static Web App Preview

Environment Deployment
pr1951 View workflow run

Deployed from commit 011dabc956fab606f49bf82b9b8bc94687f26297 via the publisher workflow.

@Reshrahim Reshrahim changed the base branch from v0.58 to v0.59 July 10, 2026 16:43
Comment thread docs/layouts/partials/hooks/body-end.html
Comment thread docs/config.toml Outdated
Comment thread docs/config.toml Outdated
Comment thread docs/config.toml Outdated
Co-authored-by: Reshma Abdul Rahim <61033581+Reshrahim@users.noreply.github.com>
Signed-off-by: Reshma Abdul Rahim <61033581+Reshrahim@users.noreply.github.com>
Comment thread docs/config.toml
Co-authored-by: Reshma Abdul Rahim <61033581+Reshrahim@users.noreply.github.com>
Signed-off-by: Reshma Abdul Rahim <61033581+Reshrahim@users.noreply.github.com>
The _build front matter key was removed in Hugo 0.145.0; the CI uses
hugo-extended 0.161.1, so switch to the build key (with cascade) to keep
the installation shared-content section hidden from rendering and nav.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: e412b758-7998-4133-ab00-e904a086f732
Signed-off-by: Reshma Abdul Rahim <reshmarahim.abdul@microsoft.com>
…ation

Radius core now ships Radius.Data/postgreSqlDatabases in the built-in
radius extension, which collided with the tutorial's radiusResources
extension (BCP264). Remove the redundant radiusResources import from the
snippet and update the prose accordingly.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: e412b758-7998-4133-ab00-e904a086f732
Signed-off-by: Reshma Abdul Rahim <reshmarahim.abdul@microsoft.com>

@DariuszPorowski DariuszPorowski left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@Reshrahim I think you should choose edge branch as target

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