Skip to content
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

Update staking API client for updated openapi spec #316

Merged
merged 4 commits into from
Nov 20, 2024

Conversation

rohan-agarwal-coinbase
Copy link
Contributor

What changed? Why?

  • OpenAPI spec changed from Wallet tag to MPCWallet and Validator was migrated to Stake. This PR updates the clients and tests accordingly

Testing

Unit tests pass

Qualified Impact

@drohit-cb
Copy link
Contributor

Thanks @rohan-agarwal-coinbase for doing this! Appreciate it 🙏

@rohan-agarwal-coinbase rohan-agarwal-coinbase merged commit c8bb9cc into v0.11.0 Nov 20, 2024
4 checks passed
@rohan-agarwal-coinbase rohan-agarwal-coinbase deleted the rohan/fix-staking branch November 20, 2024 15:55
@alex-stone alex-stone mentioned this pull request Nov 27, 2024
alex-stone added a commit that referenced this pull request Nov 27, 2024
* Update staking API client for updated openapi spec (#316)

* Fix Asset fromModel bug to use passed in asset_id when initializing (#317)

* feat: update client to include rlp_encoded_tx field within EthereumTransaction

* add staking e2e tests

* feat: Add source & source version attribution

This adds the source & source version attribution handling.

This makes it so that when configuring the SDK, the `source` and
`sourceVersion` can be configured and passed through as correlation
context for attribution on the backend.

This will make it so we can easily attribute AgentKit requests to
the specific version.

* Support for fund and quote_fund for wallet funding (#322)

* chore: Add source/source version to configure from JSON

* chore: update stake build test to use mainnet to reduce falkiness observed on testnet

* chore: Prep v0.11.0 release

---------

Co-authored-by: rohan-agarwal-coinbase <[email protected]>
Co-authored-by: Rohit Durvasula <[email protected]>
Co-authored-by: Rohit Durvasula <[email protected]>
Co-authored-by: Howard Xie <[email protected]>
Co-authored-by: cb-howardatcb <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants