-
Notifications
You must be signed in to change notification settings - Fork 39
Teach CLI the get rates command #748
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
Conversation
WalkthroughA new "get-rates" command has been introduced to the CLI application. This command enables users to fetch and display current rates from the rates manager contract. The implementation involves adding a Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant CLI
participant BlockchainClient
participant ContractRatesFetcher
User->>CLI: Execute "get-rates" command
CLI->>BlockchainClient: Create client with 15s deadline
CLI->>ContractRatesFetcher: Instantiate fetcher
CLI->>ContractRatesFetcher: Start fetcher
alt No rates found
ContractRatesFetcher-->>CLI: Return "no rates found" error
CLI->>CLI: Log info, exit early
else Rates available
ContractRatesFetcher-->>CLI: Return current rates
CLI->>CLI: Log/display rates
end
Possibly related PRs
Suggested reviewers
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 golangci-lint (1.64.8)Error: you are using a configuration file for golangci-lint v2 with golangci-lint v1: please use golangci-lint v2 📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (2)
⏰ Context from checks skipped due to timeout of 90000ms (1)
🔇 Additional comments (8)
✨ Finishing Touches
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. 🪧 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 using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
Add
get-ratescommand to CLI to retrieve current rates from rates managerImplements a new CLI command
get-rateswith supporting infrastructure:GetRatesOptionsstruct in cliOptions.gogetRatesfunction that creates and starts a contract rates fetcher to retrieve current ratesadd-rateswith other admin commands for better organization📍Where to Start
Start with the
getRatesfunction implementation in main.go which contains the core logic for fetching rates using the contract rates fetcher.Macroscope summarized a947cbd.
Summary by CodeRabbit