Skip to content

fix(nav): hide Memory tab from bottom navigation bar#992

Merged
graycyrus merged 1 commit into
tinyhumansai:mainfrom
graycyrus:fix/hide-memory-tab
Apr 27, 2026
Merged

fix(nav): hide Memory tab from bottom navigation bar#992
graycyrus merged 1 commit into
tinyhumansai:mainfrom
graycyrus:fix/hide-memory-tab

Conversation

@graycyrus
Copy link
Copy Markdown
Contributor

@graycyrus graycyrus commented Apr 27, 2026

Summary

  • Hides the Memory tab from the bottom navigation bar until the Intelligence feature is ready for release
  • The /intelligence route and page remain intact — only the nav entry is commented out

Test plan

  • Verify the Memory tab no longer appears in the bottom navigation bar
  • Verify all other tabs (Home, Chat, Connections, Alerts, Rewards, Settings) still render and navigate correctly
  • Verify /intelligence route still works if accessed directly via URL

Closes #976

Summary by CodeRabbit

  • Bug Fixes
    • Removed the Memory tab from the bottom navigation menu to address reported issues.

@graycyrus graycyrus requested a review from a team April 27, 2026 16:53
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 27, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: c6461c81-22ee-4479-aa43-1cf5de818b1e

📥 Commits

Reviewing files that changed from the base of the PR and between bd6a465 and efb82df.

📒 Files selected for processing (1)
  • app/src/components/BottomTabBar.tsx

📝 Walkthrough

Walkthrough

The Memory tab is removed from the bottom navigation bar by commenting out the intelligence tab entry. This change temporarily hides the Memory navigation option until the Intelligence feature is ready for release, as specified in issue #976.

Changes

Cohort / File(s) Summary
Memory Tab Removal
app/src/components/BottomTabBar.tsx
Commented out the intelligence tab entry to hide the Memory navigation option from the bottom tab bar pending feature readiness. Reference to issue #976 included in comment.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Poem

🐰 A tab hops away, tucked safe in a comment,
Memory sleeps while we gather our moment,
Soon it will wake when the time is just right—
Hidden but ready to hop back to sight! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 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: hiding the Memory tab from the bottom navigation bar, which is the primary objective of this PR.
Linked Issues check ✅ Passed The PR successfully implements the requirement from issue #976 to comment out the Memory/Intelligence tab entry in BottomTabBar.tsx while keeping the route and page intact.
Out of Scope Changes check ✅ Passed All changes are directly related to issue #976; only the navigation tab entry is modified and no unrelated changes are present in the PR.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

@graycyrus graycyrus merged commit f9c15bd into tinyhumansai:main Apr 27, 2026
9 checks passed
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.

Hide Memory tab from bottom navigation bar

1 participant