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

Support for fund and quote_fund for wallet funding #322

Merged
merged 13 commits into from
Nov 27, 2024

Conversation

rohan-agarwal-coinbase
Copy link
Contributor

What changed? Why?

  • Creates new crypto_amount and fiat_amount classes to wrap the underlying models, as well as fund_operation and fund_quote.
  • Updates wallet_address and wallet for new fund and quote_fund functions

Testing

  • Unit tests

Qualified Impact

* @returns {Decimal} The fiat amount in decimal format
*/
public getFiatAmount(): Decimal {
return new Decimal(this.model.fiat_amount.amount);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we want this to return a FiatAmount?

@rohan-agarwal-coinbase rohan-agarwal-coinbase merged commit 16508a7 into v0.11.0 Nov 27, 2024
4 checks passed
@rohan-agarwal-coinbase rohan-agarwal-coinbase deleted the rohan/PSDK-595-fund branch November 27, 2024 19:30
@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