Skip to content

Conversation

@rohan-agarwal-coinbase
Copy link
Contributor

What changed? Why?

  • Update getDefaultAddress and getAddress to fetch addresses if not loaded. The change to the interfaces are backward-incompatible, which is why we bumped the minor version

…ded (#217)

* Update getDefaultAddress and getAddress to fetch addresses if not loaded

* Address feedback

* lint

* WIP

* Fix e2e test

* Fix

* 1 failing test

* Cleanup

* Fix

* more test coverage

* Address feedback

* Improve

* Update jest config
@cb-heimdall
Copy link

cb-heimdall commented Sep 6, 2024

✅ Heimdall Review Status

Requirement Status More Info
Reviews 1/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 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

@rohan-agarwal-coinbase rohan-agarwal-coinbase merged commit 2e5ff3e into master Sep 7, 2024
@jazz-cb jazz-cb deleted the v0.4.0 branch September 7, 2024 00:07
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.

4 participants