-
-
Notifications
You must be signed in to change notification settings - Fork 137
Fixed (breadcrumb): ensure breadcrumb UI updates when folderId is removed #119
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
Conversation
Previously, clicking the home button removed the folderId and updated the data, but the breadcrumb UI didn't reflect the change. Updated the logic to ensure the breadcrumb component re-renders correctly when navigating home.
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the WalkthroughThis update modifies setup documentation and internal logic in several frontend components. The README now points to the correct environment file. UI code gains a guard in the dashboard header, an import is clarified in the home hero, and a conditional query option is removed from a custom hook. Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant FileBreadcrumb (Dashboard Header)
participant Router
User->>FileBreadcrumb: Click "Home"
FileBreadcrumb->>FileBreadcrumb: Check if folderId exists
alt folderId exists
FileBreadcrumb->>Router: Update URL (remove folderId)
else folderId absent
FileBreadcrumb-->>User: Do nothing
end
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Poem
✨ Finishing Touches🧪 Generate unit tests
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
59dfa94
to
d6881ae
Compare
Sorry for my hiatus. This can be fixed by changing Thanks for the contribution 😊 |
Pull Request
Description
Updated the logic to ensure the breadcrumb component re-renders correctly when navigating home.
Currently, Clicking the home button removed the folderId from search params and updated the data, but the breadcrumb UI didn't reflect the change.
Screencast.from.2025-08-02.02-13-24.webm
Type of Change
Changes
enabled
option from useQuery hooks for cleaner logic.Screenshots/Videos
Testing
Checklist
Summary by CodeRabbit
Bug Fixes
Documentation
Style