Closed trajan0x closed 2 weeks ago
[!WARNING]
Rate limit exceeded
@github-actions[bot] has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 19 minutes and 58 seconds before requesting another review.
⌛ How to resolve this issue?
After the wait time has elapsed, a review can be triggered using the `@coderabbitai review` command as a PR comment. Alternatively, push new commits to this PR. We recommend that you space out your commits to avoid hitting the rate limit.🚦 How do rate limits work?
CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our [FAQ](https://coderabbit.ai/docs/faq) for further information.📥 Commits
Reviewing files that changed from the base of the PR and between 36b6345912c93e26c046985c171e1fd46ae56b77 and a750e89f43dfa15d02c4a7defe16c1b59d1039c5.
The pull request introduces a new dependency, @ethersproject/abi
, to the package.json
of the @synapsecns/rest-api
project. It modifies the bridgeController.ts
file to enhance bridging operations by adding a new constant for the bridge interface and updating the payload structure to include encoded transaction data. Additionally, the bridgeRoute.ts
file is updated to include a new txData
property in the response schema for the /bridge
endpoint, providing encoded transaction calldata.
File | Change Summary |
---|---|
packages/rest-api/package.json | Added dependency: "@ethersproject/abi": "^5.7.0" |
packages/rest-api/src/controllers/bridgeController.ts | Added const BRIDGE_INTERFACE and updated payload to include txData for the bridge function. |
packages/rest-api/src/routes/bridgeRoute.ts | Updated response schema for /bridge endpoint to include txData property of type string . |
bridgeController.ts
regarding the calculation of bridgeFeeFormatted
may relate to the new functionality introduced in the main PR, which enhances the bridging operations by adding a new dependency for handling ABI encoding.BRIDGE_MAP
and bridgeable.ts
files to include new tokens and modify existing ones are relevant as they expand the bridging capabilities, which aligns with the enhancements made in the main PR.bridgeMap
and bridgeable.ts
files is directly related to the bridging functionality being enhanced in the main PR, which also involves updates to the bridging operations.M-deps
, Sol
, Typescript
, size/xs
🐰 In the code, I hop and play,
With bridges built to save the day.
A new dependency, oh what a treat,
Encoding data, can't be beat!
WithtxData
now in sight,
Our transactions take to flight! 🌟
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?
Description A clear and concise description of the features you're adding in this pull request.
Additional context Add any other context about the problem you're solving.
Metadata
Summary by CodeRabbit
New Features
txData
field in the/bridge
endpoint response, providing encoded transaction data.Dependencies
@ethersproject/abi
version^5.7.0
.