Feature/update link - #104
Merged
Merged
Conversation
#97) (#101) * feat: replace Partners tab with Verticals dropdown and remove partners page - Navbar Partners tab is now a Verticals dropdown (desktop + mobile) linking to TradeTrust, OpenCerts, and SAL, opening in new tabs - Remove /partners route and PartnersSection from Home - Pin @trustvc/trustvc to 2.15.0-beta.2 in lockfile * test: update navbar and home tests for Verticals dropdown - Navbar/App tests: expect Verticals instead of removed Partners tab - Add Verticals dropdown test asserting TradeTrust/OpenCerts/SAL links - Drop Home Partners-section tests (section removed) * fix: keep mobile Verticals submenu interactions contained on outside-click The document mousedown handler closed the Verticals dropdown whenever the target was outside the desktop dropdown container, so taps on the mobile submenu (links and toggle) unmounted the target before click fired. Track the mobile menu with its own ref and treat clicks inside either container as contained. --------- Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
Contributor
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (9)
📝 WalkthroughWalkthroughThe navbar replaces Partners with responsive Verticals menus linking to TradeTrust, OpenCerts, and SAL. The Partners route and home section are removed. TrustVC documentation links now use ChangesVerticals navigation
Estimated code review effort: 3 (Moderate) | ~20 minutes Sequence Diagram(s)sequenceDiagram
participant Visitor
participant Navbar
participant ExternalSite
Visitor->>Navbar: Open Verticals menu
Navbar-->>Visitor: Show external links
Visitor->>Navbar: Select a vertical
Navbar->>ExternalSite: Open link in a new tab
Navbar-->>Visitor: Close menu
Possibly related PRs
Suggested reviewers: ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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. Comment |
RishabhS7
approved these changes
Aug 12, 2026
rongquan1
pushed a commit
that referenced
this pull request
Aug 13, 2026
* feat: replace Partners tab with Verticals dropdown and remove partner… (#97) * feat: replace Partners tab with Verticals dropdown and remove partners page - Navbar Partners tab is now a Verticals dropdown (desktop + mobile) linking to TradeTrust, OpenCerts, and SAL, opening in new tabs - Remove /partners route and PartnersSection from Home - Pin @trustvc/trustvc to 2.15.0-beta.2 in lockfile Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> * test: update navbar and home tests for Verticals dropdown - Navbar/App tests: expect Verticals instead of removed Partners tab - Add Verticals dropdown test asserting TradeTrust/OpenCerts/SAL links - Drop Home Partners-section tests (section removed) Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> * fix: keep mobile Verticals submenu interactions contained on outside-click The document mousedown handler closed the Verticals dropdown whenever the target was outside the desktop dropdown container, so taps on the mobile submenu (links and toggle) unmounted the target before click fired. Track the mobile menu with its own ref and treat clicks inside either container as contained. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> --------- Co-authored-by: Claude Fable 5 <noreply@anthropic.com> * Feature/boe v2 (#100) * feat: added integration for the trustvc sdk for the BOE beta version * feat: updated package json * feat: udpate the bligation functions for etr * feat: fix coderabbit issues * feat: integrate obligation handling in AssetManagement components - Added isObligation prop to DocumentStatus and AssetManagementTags components to manage obligation-specific UI behavior. - Enhanced ActionSelectionForm to display obligation status labels and manage visibility based on isObligation state. * feat: fix overflow of the dropdown container * chore: update @trustvc/trustvc and related dependencies to version 2.16.0-beta.3 and 2.4.2 in package.json and package-lock.json * chore: update @trustvc/trustvc and related dependencies to version 2.16.0-beta.3 and 2.4.2 in package.json and package-lock.json (#102) * chore: update package-lock.json to include new dependencies for @trustvc/trustvc * Feature/update link (#104) * feat: replace Partners tab with Verticals dropdown and remove partner… (#97) (#101) * feat: replace Partners tab with Verticals dropdown and remove partners page - Navbar Partners tab is now a Verticals dropdown (desktop + mobile) linking to TradeTrust, OpenCerts, and SAL, opening in new tabs - Remove /partners route and PartnersSection from Home - Pin @trustvc/trustvc to 2.15.0-beta.2 in lockfile * test: update navbar and home tests for Verticals dropdown - Navbar/App tests: expect Verticals instead of removed Partners tab - Add Verticals dropdown test asserting TradeTrust/OpenCerts/SAL links - Drop Home Partners-section tests (section removed) * fix: keep mobile Verticals submenu interactions contained on outside-click The document mousedown handler closed the Verticals dropdown whenever the target was outside the desktop dropdown container, so taps on the mobile submenu (links and toggle) unmounted the target before click fired. Track the mobile menu with its own ref and treat clicks inside either container as contained. --------- Co-authored-by: Claude Fable 5 <noreply@anthropic.com> * feat: update links --------- Co-authored-by: isaackps <isaac.kps.work@gmail.com> Co-authored-by: Claude Fable 5 <noreply@anthropic.com> --------- Co-authored-by: isaackps <isaac.kps.work@gmail.com> Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary by CodeRabbit
New Features
Bug Fixes
Removals