Skip to content

Fix Data Access missing from docs side panel product dropdown#3659

Merged
Baalmart merged 1 commit into
stagingfrom
fix-data-nav
Jun 17, 2026
Merged

Fix Data Access missing from docs side panel product dropdown#3659
Baalmart merged 1 commit into
stagingfrom
fix-data-nav

Conversation

@Baalmart

@Baalmart Baalmart commented Jun 17, 2026

Copy link
Copy Markdown
Collaborator

🚀 Pull Request

📋 Description

What does this PR do?

Adds "Data Access" as a selectable option in the ProductDropdown component used in the docs side panel. The AqDatabase01 icon and the /docs/data-access/researchers-guide landing path are wired up alongside the existing products.

Why is this change needed?

When navigating to any /docs/data-access/* page (e.g. the Researcher's Guide), the side panel product selector had no matching entry for data-access, so it fell back to the first item in the list — "Analytics" — and displayed that as the active product instead of "Data Access".


🔗 Related Issues

  • Closes #
  • Fixes #
  • Related to #

🔄 Type of Change

  • 🐛 Bug fix
  • ✨ New feature
  • 🔧 Enhancement/improvement
  • 📚 Documentation update
  • ♻️ Refactor
  • 🗑️ Removal/deprecation

🏗️ Affected Services

Microservices changed:

  • src/docs-websitesrc/components/ProductDropdown/index.tsx

🧪 Testing

  • Unit tests added/updated
  • Manual testing completed
  • All existing tests pass

Test summary:
Navigated to /docs/data-access/researchers-guide/ and confirmed the side panel now shows "Data Access" as the active selection instead of "Analytics". Verified all other product routes (/docs/analytics, /docs/vertex, etc.) still resolve to their correct entries.


💥 Breaking Changes

  • No breaking changes
  • Has breaking changes (describe below)

📝 Additional Notes

The AqDatabase01 icon imported here was already used for the Data Access card in ProductSection/index.tsx, keeping the icon consistent between the home page grid and the side panel dropdown.


✅ Checklist

  • Code follows project style guidelines
  • Self-review completed
  • Documentation updated (if needed)
  • Ready for review

Summary by CodeRabbit

Release Notes

  • New Features
    • Added a new "Data Access" option to the product dropdown menu, providing users with quick access to documentation and dedicated landing page resources.

@coderabbitai

coderabbitai Bot commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: f81555ed-6ac3-4b38-9967-b649898ea73e

📥 Commits

Reviewing files that changed from the base of the PR and between 10fb029 and 14f0249.

📒 Files selected for processing (1)
  • src/docs-website/src/components/ProductDropdown/index.tsx

📝 Walkthrough

Walkthrough

A new "Data Access" product entry is added to the ProductDropdown component. The AqDatabase01 icon is imported, and the PRODUCTS constant is extended with the "Data Access" entry, its documentation path, and a landingPath used for navigation on selection.

Changes

Data Access Entry in ProductDropdown

Layer / File(s) Summary
Icon import and Data Access product registration
src/docs-website/src/components/ProductDropdown/index.tsx
AqDatabase01 is added to the icon imports; a new PRODUCTS entry for "Data Access" is registered with path, icon, and landingPath.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • airqo-platform/AirQo-frontend#3610: Also adds a "Data Access" product entry using the AqDatabase01 icon and the same /data-access/... routes, but targets ProductSection on the docs homepage rather than ProductDropdown.

Poem

A dropdown grows, a new leaf on the tree,
AqDatabase01 arrives, icon proud and free,
"Data Access" joins the list, paths set just right,
One small entry, navigation shining bright. 🗄️✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: adding a missing 'Data Access' entry to the ProductDropdown component in the docs side panel.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix-data-nav

Warning

Review ran into problems

🔥 Problems

Errors were encountered while retrieving linked issues.

Errors (1)
  • JIRA integration encountered authorization issues. Please disconnect and reconnect the integration in the CodeRabbit UI.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes the docs sidebar product selector by adding “Data Access” to the ProductDropdown, ensuring /docs/data-access/* routes correctly map to a matching product instead of falling back to “Analytics”.

Changes:

  • Added AqDatabase01 icon import and a new PRODUCTS entry for Data Access.
  • Wired the Data Access product to /docs/data-access with landing route /docs/data-access/researchers-guide.

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

@github-actions

Copy link
Copy Markdown
Contributor

New azure docs changes available for preview here

@Baalmart
Baalmart merged commit a974fe1 into staging Jun 17, 2026
28 checks passed
@Baalmart
Baalmart deleted the fix-data-nav branch June 17, 2026 10:19
@Baalmart Baalmart mentioned this pull request Jun 17, 2026
1 task
@coderabbitai coderabbitai Bot mentioned this pull request Jul 7, 2026
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants