-
Notifications
You must be signed in to change notification settings - Fork 33
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
[DO NOT MERGE] test/sdk-rfq #2914
Conversation
Important Review skippedIgnore keyword(s) in the title. Ignored keywords (2)
Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configuration File (
|
Deploying sanguine-fe with
|
Latest commit: |
c40b618
|
Status: | ✅ Deploy successful! |
Preview URL: | https://1108a8d8.sanguine-fe.pages.dev |
Branch Preview URL: | https://test-sdk-rfq.sanguine-fe.pages.dev |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR Summary
This pull request introduces several updates to support new blockchain networks and tokens, along with some debugging enhancements.
- Debugging: Added
console.log
inpackages/sdk-router/src/operations/bridge.ts
for route inspection duringallBridgeQuotes
execution. - UI Update: Modified
packages/synapse-interface/components/layouts/LandingPageWrapper/index.tsx
to update theAnnouncementBanner
with new RFQ transaction information. - Token Support: Updated
packages/synapse-interface/constants/bridgeMap.ts
to include RFQ.USDC and RFQ.ETH tokens. - Blockchain Support: Added Linea chain ID (59144) in
packages/synapse-interface/constants/chains/index.tsx
and integrated Linea network inpackages/synapse-interface/constants/chains/master.tsx
. - Dependency Management: Changed
packages/synapse-interface/package.json
to use a local version of@synapsecns/sdk-router
for development purposes.
10 file(s) reviewed, 1 comment(s)
Edit PR Review Bot Settings
@@ -141,6 +141,8 @@ export async function allBridgeQuotes( | |||
tokenOut, | |||
amountIn | |||
) | |||
console.log('routes: ', routes) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Style: Remove or replace this console.log statement with proper logging before merging to master.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR Summary
(updates since last review)
Recent changes focus on enhancing token routing and dependency management, along with adding debugging capabilities.
- Token Routing: Updated
packages/synapse-interface/constants/bridgeMap.ts
to include 'RFQ.USDB' and 'RFQ.USDC' for USDC and USDB tokens. - Dependency Management: Updated
packages/synapse-interface/package.json
to use@synapsecns/sdk-router
version^0.9.0
from a package registry. - Debugging: Added test calls and logging for
synapseSDK.allBridgeQuotes
inpackages/synapse-interface/pages/state-managed-bridge/index.tsx
.
These changes improve token route flexibility, ensure stable dependency usage, and enhance debugging capabilities.
3 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #2914 +/- ##
====================================================
+ Coverage 25.71470% 90.49505% +64.78035%
====================================================
Files 770 54 -716
Lines 55513 1010 -54503
Branches 80 80
====================================================
- Hits 14275 914 -13361
+ Misses 39760 93 -39667
+ Partials 1478 3 -1475
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
null
8954400: synapse-interface preview link
5f4daa3: synapse-interface preview link