-
Notifications
You must be signed in to change notification settings - Fork 621
Dashboard: Hide cross-chain tab on non-modular contract pages #7950
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
Dashboard: Hide cross-chain tab on non-modular contract pages #7950
Conversation
|
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the WalkthroughAdds conditional hiding of the “Cross Chain (Beta)” sidebar link based on Modular Core status and introduces an early redirect guard in the cross-chain page to route non-Modular Core contracts back to the contract landing page. Changes
Sequence Diagram(s)sequenceDiagram
autonumber
actor User
participant Page as CrossChainPage
participant Checker as ModularCoreCheck
participant Router as Redirector
participant Landing as ContractLandingPage
User->>Page: Navigate to Cross Chain (Beta)
Page->>Checker: Determine isModularCore
alt isModularCore == false
Page->>Router: redirectToContractLandingPage(chainIdOrSlug, contractAddress, projectMeta)
Router-->>User: HTTP Redirect
User->>Landing: Load landing page
else isModularCore == true
Note right of Page: Proceed with existing init<br/>(clone factory, code fetch,<br/>cross-chain metadata)
end
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes ✨ Finishing Touches🧪 Generate unit tests
🪧 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/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
How to use the Graphite Merge QueueAdd either label to this PR to merge it via the merge queue:
You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has enabled the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. This stack of pull requests is managed by Graphite. Learn more about stacking. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #7950 +/- ##
=======================================
Coverage 56.53% 56.53%
=======================================
Files 904 904
Lines 58623 58623
Branches 4146 4146
=======================================
Hits 33144 33144
Misses 25373 25373
Partials 106 106
🚀 New features to boost your workflow:
|
d795415 to
dac2c48
Compare
size-limit report 📦
|

PR-Codex overview
This PR focuses on enhancing the sidebar links and layout for a contract page, particularly introducing conditional rendering based on the
isModularCoreproperty, which affects the visibility of certain features.Detailed summary
hideproperty to the sidebar link for "Cross Chain (Beta)" based ondata.metadata.isModularCore.contract-page-layout.tsxto include aflexclass for better styling.shared-cross-chain-page.tsxifisModularCoreis false.Summary by CodeRabbit