Closed golangisfun123 closed 3 days ago
[!IMPORTANT]
Review skipped
Auto reviews are disabled on base/target branches other than the default branch.
Base branches to auto review (3)
* feat/.* * master * fe-releasePlease check the settings in the CodeRabbit UI or the
.coderabbit.yaml
file in this repository. To trigger a single review, invoke the@coderabbitai review
command.You can disable this status message by setting the
reviews.review_status
tofalse
in the CodeRabbit configuration file.
This update implements several enhancements across the ethergo
and services/rfq
packages. Key changes include updating constants, adding new methods to interfaces, expanding API capabilities to handle token withdrawals, modifying configurations and handlers to support these new functions, and introducing new command-line tools for these operations. Comprehensive tests ensure robustness and reliability.
File(s) | Change Summary |
---|---|
ethergo/submitter/submitter.go |
Updated constant and added Address() method to TransactionSubmitter interface. |
services/rfq/relayer/relapi/client.go |
Added Withdraw method to RelayerClient interface and implemented it. |
services/rfq/relayer/relapi/handler.go |
Enhanced Handler struct, added Withdraw function, helper functions, new imports, and WithdrawRequest struct. |
services/rfq/relayer/relapi/handler_test.go |
Added test for marshaling and unmarshaling WithdrawRequest struct. |
services/rfq/relayer/relapi/server.go |
Modified RelayerAPIServer to include submitter and added a POST route for withdrawals. |
services/rfq/relayer/relapi/server_test.go |
Added tests for Ethereum and ERC20 token withdrawals. |
services/rfq/relayer/relapi/suite_test.go |
Expanded RelayerServerSuite with new imports, variables, and additional setup logic. |
services/rfq/relayer/relconfig/config.go |
Added new fields to Config struct for withdrawal functionalities and helper function DecodeTokenID . |
services/rfq/tools/withdrawal/cmd/cmd.go |
Introduced command-line tool for handling withdrawal processes. |
services/rfq/tools/withdrawal/cmd/commands.go |
Defined configuration flags and command for running the withdrawal tool. |
services/rfq/tools/withdrawal/withdraw/withdraw_test.go |
Added package declaration for testing withdrawal processes. |
contrib/opbot/config/config.go |
Modified SignozBaseURL field and added comment for RelayerURLS in Config struct. |
sequenceDiagram
participant Client
participant RelayerAPI
participant RelayerServer
participant Submitter
participant Blockchain
Client->>RelayerAPI: POST /withdraw (withdrawRequest)
RelayerAPI->>RelayerServer: Handle withdraw request
RelayerServer->>Submitter: Submit transaction (submit)
Submitter->>Blockchain: Send transaction
Blockchain->>Submitter: Transaction receipt
Submitter->>RelayerServer: Transaction success
RelayerServer->>RelayerAPI: Return withdraw response
RelayerAPI->>Client: WithdrawResponse (nonce)
In fields of code, where changes bloom,
We crafted features, lifted gloom.
Withdrawals flow with tokens bright,
API and CLI ignite.
Tests ensure the paths are clear,
For users far and engineers near.
Rejoice, for updates hold the key,
To seamless blockchain harmony! 🌿
[!TIP]
Early access features: enabled
We are currently testing the following features in early access: - **OpenAI `gpt-4o` model for code reviews and chat**: OpenAI claims that this model is better at understanding and generating code than the previous models. We seek your feedback over the next few weeks before making it generally available. Note: - You can enable or disable early access features from the CodeRabbit UI or by updating the CodeRabbit configuration file. - Please join our [Discord Community](https://discord.com/invite/GsXnASn26c) to provide feedback and report issues. - OSS projects are currently opted into early access features by default.
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?
Latest commit: |
97634de
|
Status: | ✅ Deploy successful! |
Preview URL: | https://6be918b5.sanguine-fe.pages.dev |
Branch Preview URL: | https://rfq-api-cli.sanguine-fe.pages.dev |
Attention: Patch coverage is 0%
with 42 lines
in your changes missing coverage. Please review.
Project coverage is 25.72228%. Comparing base (
484a9a3
) to head (c4b5ce5
).
Files | Patch % | Lines |
---|---|---|
services/rfq/relayer/cmd/commands.go | 0.00000% | 36 Missing :warning: |
services/rfq/relayer/relapi/handler.go | 0.00000% | 5 Missing :warning: |
services/rfq/relayer/cmd/cmd.go | 0.00000% | 1 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
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
Withdraw
method and correspondingWithdrawRequest
andWithdrawResponse
structs.Enhancements
RelayerAPIServer
andHandler
structures to support the new withdrawal functionalities.Configuration