Skip to content

Enable Tier 2 secondary navigation band#859

Draft
ppiegaze wants to merge 3 commits intomainfrom
peeter/tier2-band
Draft

Enable Tier 2 secondary navigation band#859
ppiegaze wants to merge 3 commits intomainfrom
peeter/tier2-band

Conversation

@ppiegaze
Copy link
Collaborator

Summary

  • Enables the new Tier 2 secondary navigation band on three key sections by adding tier2_band: true frontmatter
  • Updates unionai-docs-infra submodule to PR #51 which implements the Tier 2 feature

Sections enabled

  • User guide — subsections (Core concepts, Configure tasks, Build tasks, etc.) appear as Tier 2 tabs
  • Reference — subsections (Flyte SDK, Flyte CLI, Migration, etc.) appear as Tier 2 tabs
  • Platform deployment — subsections appear as Tier 2 tabs (variant-filtered)

Other sections (Tutorials, Integrations, Community, Release notes) are unchanged — their children remain in the sidebar.

Dependencies

Test plan

  • make dev — verify Tier 2 band appears on User guide, Reference, Platform deployment
  • Verify Tier 2 band does NOT appear on Tutorials, Integrations, etc.
  • Click Tier 2 tabs — verify sidebar shows only the active child's subtree
  • make dist — all variants build without errors

🤖 Generated with Claude Code

Copilot AI review requested due to automatic review settings March 19, 2026 07:46
Copy link
Contributor

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

Enables the new Tier 2 secondary navigation band for selected site sections and updates the docs-infra submodule to the commit that implements the feature.

Changes:

  • Enable Tier 2 band via tier2_band: true frontmatter on User guide, Reference, and Platform deployment index pages.
  • Bump unionai-docs-infra submodule to a newer commit containing Tier 2 support.

Reviewed changes

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

File Description
unionai-docs-infra Updates infra submodule pointer to include Tier 2 navigation support.
content/user-guide/_index.md Turns on Tier 2 band for User guide section.
content/deployment/_index.md Turns on Tier 2 band for Platform deployment section.
content/api-reference/_index.md Turns on Tier 2 band for Reference section.
Comments suppressed due to low confidence (1)

unionai-docs-infra:1

  • The PR description indicates this should be updated after unionai-docs-infra#51 is merged, but the submodule is pinned to a raw commit hash. To reduce risk of the main repo depending on an unmerged/rewritten commit, update the submodule to a commit that is reachable from unionai-docs-infra’s default branch (post-merge) or to a stable tag/release commit if your workflow supports it.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Mar 19, 2026

Deploying docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: f43a83a
Status: ✅  Deploy successful!
Preview URL: https://920384e8.docs-dog.pages.dev
Branch Preview URL: https://peeter-tier2-band.docs-dog.pages.dev

View logs

@ppiegaze ppiegaze marked this pull request as draft March 19, 2026 07:50
@ppiegaze ppiegaze force-pushed the peeter/tier2-band branch from 362d96d to 4ac182c Compare March 19, 2026 13:53
ppiegaze and others added 3 commits March 25, 2026 07:36
Structural changes from IA review (ia-review-deferred.md items 1-8, 11):

- Create getting-started/ section: moved overview, quickstart, running-locally,
  connecting-to-a-cluster, projects-and-domains, authenticating, user-management
- Create tutorials/ section: moved basic-project and advanced-project/
- Move CLI ref pages (run-command-options, deploy-command-options) to api-reference/
- Add subsections to task-programming: data-types/, execution-patterns/, development/
- Merge apps sections (configure-apps, build-apps, serve-and-deploy-apps) into apps/
- Group build-agent/ and sandboxing/ into agentic-ai/
- Fix all weight collisions in task-programming
- Move flyte-2/ to end of guide (weight 21)
- Update all cross-references (31 links across 18 files)

Build passes clean (make dist).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
IA review item #13: the parameter tables and method docs were
duplicated from the generated API reference. Link instead.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…ing into tasks/

Option B full restructuring: consolidates four separate task-related sections
into a single tasks/ super-section with six subsections:

  tasks/
    configuring/         (was task-configuration)
    data-types/          (from task-programming)
    execution-patterns/  (from task-programming)
    development/         (from task-programming)
    deploying/           (was task-deployment)
    scaling/             (was run-scaling)

Updated ~75 cross-references across the entire content tree.
Build passes clean (make dist).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@ppiegaze ppiegaze force-pushed the peeter/tier2-band branch from c1ac12b to f43a83a Compare March 25, 2026 14:38
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.

2 participants