Skip to content

Website Draft: Remove community landing page, go to Contribute - #331

Merged
aidanfnv merged 4 commits into
dev/aidanf/flashdreams-sitefrom
dev/aidanf/flashdreams_site/community-index-contribute
Jun 11, 2026
Merged

Website Draft: Remove community landing page, go to Contribute#331
aidanfnv merged 4 commits into
dev/aidanf/flashdreams-sitefrom
dev/aidanf/flashdreams_site/community-index-contribute

Conversation

@aidanfnv

@aidanfnv aidanfnv commented Jun 11, 2026

Copy link
Copy Markdown
Collaborator

Removes the dedicated community/contribute.rst page, which just includes CONTRIBUTE.md, replaces the content of the Community landing page community/index.rst with an include of CONTRIBUTE.md, and renames "Community" to "Contribute".

The other community/* pages are still accessible in the TOC sidebar in the community/* pages.

@copy-pr-bot

copy-pr-bot Bot commented Jun 11, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@aidanfnv
aidanfnv marked this pull request as ready for review June 11, 2026 20:34
@greptile-apps

greptile-apps Bot commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR collapses the two-page Community/Contribute structure into a single page: community/index now directly includes CONTRIBUTING.md and is renamed "Contribute" throughout the site nav.

  • community/contribute.rst is deleted; its .. include:: ../../../CONTRIBUTING.md directive is moved verbatim into community/index.rst, following an established pattern in the codebase.
  • All internal :doc:contribute references in `faq.rst` and `support.rst` are updated to `:doc:`/community/index, and the community/index overrides in conf.py are removed so the page picks up the standard community/* sidebar/TOC treatment.
  • The _fd_is_reference condition in layout.html is simplified to pagename.startswith('community/') without exceptions, consistently placing community/index in the reference-doc scaffold.

Confidence Score: 5/5

Safe to merge — purely a documentation restructuring with no functional code changes.

All internal cross-references are correctly updated, the deleted page's content is faithfully re-homed in the new location, and the layout.html routing change is mechanically correct. The two stale items in layout.html (block comment and _fd_hero_indices) were already called out in a previous review round and are cosmetic.

docs/source/_templates/layout.html — the block comment and _fd_hero_indices entry noted in the previous review are still present.

Important Files Changed

Filename Overview
docs/source/_templates/layout.html Simplified _fd_is_reference condition to cover all community/* pages (including community/index); block comment and _fd_hero_indices entry are now stale (flagged in previous review).
docs/source/community/index.rst Hero landing content replaced with a bare toctree + .. include:: ../../../CONTRIBUTING.md; contribute entry removed from toctree. Pattern mirrors the now-deleted contribute.rst.
docs/source/community/contribute.rst File deleted; its sole content (.. include:: ../../../CONTRIBUTING.md) is now inlined into community/index.rst.
docs/source/conf.py Removed community/index overrides in secondary_sidebar_items and html_sidebars, letting the community/* wildcard apply uniformly; community/index now gets page-toc and sidebar-nav-bs.
docs/source/index.rst CTA button label and toctree entry renamed from "Community" to "Contribute"; no functional change to routing.
docs/source/community/faq.rst :doc:contribute reference updated to `:doc:`/community/index; correctly re-routes to the new landing page.
docs/source/community/support.rst Same :doc:contribute → `:doc:`/community/index update as faq.rst; no other changes.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    subgraph Before
        A1[community/index\nHero landing page\nmarketing layout] --> B1[community/contribute\nIncludes CONTRIBUTING.md\nreference layout]
        A1 --> C1[community/support]
        A1 --> D1[community/discord]
        A1 --> E1[community/faq]
    end

    subgraph After
        A2[community/index\nIncludes CONTRIBUTING.md\nreference layout] --> C2[community/support]
        A2 --> D2[community/discord]
        A2 --> E2[community/faq]
    end

    style A1 fill:#f9c,stroke:#c66
    style B1 fill:#f9c,stroke:#c66
    style A2 fill:#9cf,stroke:#66c
Loading

Reviews (2): Last reviewed commit: "Merge branch 'dev/aidanf/flashdreams-sit..." | Re-trigger Greptile

…s_site/community-index-contribute

Signed-off-by: aidanfnv <aidanf@nvidia.com>
@aidanfnv aidanfnv changed the title Website: Remove community landing page, go to Contribute Website Draft: Remove community landing page, go to Contribute Jun 11, 2026
@aidanfnv
aidanfnv merged commit 65808a0 into dev/aidanf/flashdreams-site Jun 11, 2026
4 checks passed
@aidanfnv
aidanfnv deleted the dev/aidanf/flashdreams_site/community-index-contribute branch June 11, 2026 22:29
aidanfnv added a commit that referenced this pull request Jun 11, 2026
* Website: Remove community landing page, go to Contribute

* Fix paths

* Revert title change

---------

Signed-off-by: aidanfnv <aidanf@nvidia.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.

1 participant