Skip to content

Move Warehouse Native to top navigation bar #2896

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 42 commits into
base: main
Choose a base branch
from

Conversation

devin-ai-integration[bot]
Copy link
Contributor

@devin-ai-integration devin-ai-integration bot commented Apr 18, 2025

Move Warehouse Native to top navigation bar and implement dynamic search filtering

This PR:

  • Moves the Warehouse Native docs to its own top-level section in the navigation bar
  • Cleans up the categories for the SDK/API section to force subsections expanded like the main sidebar
  • Ensures proper styling for both sections
  • Implements dynamic search filtering based on the active navigation section

Dynamic Search Filtering

Added a custom JavaScript solution to dynamically scope Algolia search results based on the active navigation section:

  • When in "SDKs and APIs" section, search is scoped to Client SDKs, Server SDKs, Console API, and HTTP API content
  • When in "Warehouse Native" section, search is scoped to Warehouse Native content only
  • When in main "Docs" section, search includes all content (default behavior)

The implementation directly intercepts Algolia search requests by:

  1. Detecting the current section based on URL path
  2. Adding appropriate path-based filters to Algolia search parameters
  3. Providing a visual indicator in the search modal showing which section is being searched

Cross-section links

The following links from Warehouse Native docs point to sections outside of Warehouse Native:

  • Links to experimentation sections (e.g., experiments-plus/differential-impact-detection)
  • Links to stats-engine sections (e.g., stats-engine/metric-deltas, stats-engine/variance)
  • Links to pulse sections (e.g., pulse/read-pulse, pulse/drill-down)

Testing

  • Tested locally and verified the navigation works correctly
  • Verified both SDK/API and Warehouse Native sections have proper nesting and styling
  • Tested search functionality to verify section-specific filtering works correctly:
    • When in Warehouse Native section, searching for "sdk" only shows Warehouse Native results
    • When in SDKs & APIs section, searching for terms only shows SDK/API results
    • When in Product Docs section, searching shows general documentation results

Link to Devin run: https://app.devin.ai/sessions/9d8c8e8e999947b0b292f4ecad6a956f
Requested by: [email protected]

Copy link
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@jasonwzm
Copy link
Member

jasonwzm commented Apr 18, 2025

🚀 Deployed on https://680ae37995ce9d008b016ad8--cozy-fox-0defec.netlify.app

@jasonwzm jasonwzm temporarily deployed to pull request April 18, 2025 00:09 Inactive
@jasonwzm jasonwzm temporarily deployed to pull request April 21, 2025 22:21 Inactive
@jasonwzm jasonwzm temporarily deployed to pull request April 21, 2025 22:27 Inactive
@jasonwzm jasonwzm temporarily deployed to pull request April 21, 2025 22:37 Inactive
@jasonwzm jasonwzm temporarily deployed to pull request April 21, 2025 22:41 Inactive
@jasonwzm jasonwzm temporarily deployed to pull request April 21, 2025 22:47 Inactive
@jasonwzm jasonwzm temporarily deployed to pull request April 21, 2025 22:50 Inactive
@jasonwzm jasonwzm temporarily deployed to pull request April 21, 2025 22:56 Inactive
@jasonwzm jasonwzm temporarily deployed to pull request April 21, 2025 23:03 Inactive
@jasonwzm jasonwzm temporarily deployed to pull request April 21, 2025 23:09 Inactive
@jasonwzm jasonwzm temporarily deployed to pull request April 21, 2025 23:22 Inactive
@jasonwzm jasonwzm temporarily deployed to pull request April 21, 2025 23:29 Inactive
@jasonwzm jasonwzm temporarily deployed to pull request April 25, 2025 00:39 Inactive
@jasonwzm jasonwzm temporarily deployed to pull request April 25, 2025 00:54 Inactive
@jasonwzm jasonwzm temporarily deployed to pull request April 25, 2025 00:58 Inactive
@jasonwzm jasonwzm temporarily deployed to pull request April 25, 2025 01:11 Inactive
@jasonwzm jasonwzm temporarily deployed to pull request April 25, 2025 01:17 Inactive
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