Skip to content

Conversation

@Newportgogogadit
Copy link

What changed? Why?

Qualified Impact

chaoyaji-cb and others added 30 commits August 16, 2024 09:04
comments

test and make constructor private

add types for params

npm run format
Pass address index when registering address
Cleanup staking operation + wallet address related code
This makes it so that we dynamically update networks as we update
our OpenAPI spec.

This does not add a whole network object like we have in ruby,
because we probably want to play around with how people want to
use the various objects in TS before doing that.
This should be moved to the backend similar to the ruby SDK,
however that will require a bit of a refactor around the network
objects.
…network-constants

Support dynamic network constants
This removes the user concept in favor of directly creating, listing,
importing, and fetching wallets.
This updates the docs, readmes, and changelog to reflect the
removal of the user concept.
This adds a correlation context header that includes the version
and language of the SDK.

This will be used to track what versions of the SDK developers are
using and to help reach out to upgrade versions.
Add correlation context to request headers
…ng_wallet_model

Add wallet model support for Dedicated ETH Staking
* [hotfix] patch for page setting

* add test case
rohan-agarwal-coinbase and others added 24 commits February 21, 2025 13:38
* feat: expose fee recipient and forwarded fee recipient addresses on validator and pending_claimable_balance from staking context api

* add eurc & cbbtc to Coinbase assets

* SmartWallet support (coinbase#376)

* update changelog

* Prepping v0.19.0 release (coinbase#387)

---------

Co-authored-by: Rohit Durvasula <rohit.durvasula@coinbase.com>
Co-authored-by: Rohit Durvasula <88731568+drohit-cb@users.noreply.github.com>
Co-authored-by: Milan Saxena <milan.saxena@coinbase.com>
fix: add sleep for faucet and wait for transaction history
* Improve error message when uninitialized (coinbase#394)

* feat: expose withdrawal credentials of ethereum validators

* Preparing v0.21.0 release (coinbase#397)

---------

Co-authored-by: Rohit Durvasula <rohit.durvasula@coinbase.com>
Co-authored-by: Rohit Durvasula <88731568+drohit-cb@users.noreply.github.com>
feat: add execution layer withdrawal support for native eth unstaking
coinbase#409)

* chore: fix how options are parsed when not doing native eth unstake v2

* rename function used to check for unstake operation
…nsensus-and-execution-based-unstaking

feat: unify consensus and execution based withdrawals
* Add support for execution layer validator consolidations

* chore: Prep for v0.24.0 release

* disable shared eth staking e2e test
* Remove support for Holesky (coinbase#425)

* prep for v0.25.0 release (coinbase#426)

* fix changelog date (coinbase#428)
@cb-heimdall
Copy link

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

@Newportgogogadit Newportgogogadit changed the base branch from master to add_explicit_mode July 4, 2025 16:57
Copy link
Author

@Newportgogogadit Newportgogogadit left a comment

Choose a reason for hiding this comment

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

🥰

Copy link
Author

@Newportgogogadit Newportgogogadit left a comment

Choose a reason for hiding this comment

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

License needs to 2025

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.