Skip to content
This repository has been archived by the owner on Oct 7, 2024. It is now read-only.

feat: add getAccountBalances method to Keyring #320

Merged
merged 7 commits into from
May 29, 2024
Merged

Conversation

danroc
Copy link
Contributor

@danroc danroc commented May 28, 2024

The PR adds the method that will be used to get the balances of a given account.

Fixes: https://github.com/MetaMask/accounts-planning/issues/457

@danroc danroc force-pushed the feature/add-get-balances branch 3 times, most recently from 6966f74 to 4ec30c8 Compare May 28, 2024 08:14
@danroc danroc changed the base branch from main to feature/split-api-file May 28, 2024 08:14
@danroc danroc force-pushed the feature/add-get-balances branch from 4ec30c8 to 59a1187 Compare May 28, 2024 08:22
Base automatically changed from feature/split-api-file to main May 28, 2024 08:29
@danroc danroc force-pushed the feature/add-get-balances branch 2 times, most recently from c72985f to c457fa3 Compare May 28, 2024 12:55
@danroc danroc marked this pull request as ready for review May 28, 2024 12:55
@danroc danroc requested a review from a team as a code owner May 28, 2024 12:55
@danroc danroc force-pushed the feature/add-get-balances branch 2 times, most recently from 94e72b8 to e0d1be8 Compare May 28, 2024 13:04
src/api/balance.ts Outdated Show resolved Hide resolved
src/api/keyring.ts Outdated Show resolved Hide resolved
src/internal/api.ts Show resolved Hide resolved
@danroc danroc force-pushed the feature/add-get-balances branch 3 times, most recently from fba55aa to 64b0e0f Compare May 28, 2024 20:34
@danroc danroc force-pushed the feature/add-get-balances branch from 64b0e0f to b6882da Compare May 28, 2024 20:37
@gantunesr gantunesr added the team-accounts This should be handled by the Accounts Team label May 29, 2024
src/api/keyring.ts Outdated Show resolved Hide resolved
Co-authored-by: Charly Chevalier <charly.chevalier@consensys.net>
src/internal/api.ts Outdated Show resolved Hide resolved
src/internal/api.ts Outdated Show resolved Hide resolved
src/internal/api.ts Outdated Show resolved Hide resolved
danroc and others added 5 commits May 29, 2024 10:34
Co-authored-by: Charly Chevalier <charly.chevalier@consensys.net>
Co-authored-by: Charly Chevalier <charly.chevalier@consensys.net>
Co-authored-by: Charly Chevalier <charly.chevalier@consensys.net>
@danroc danroc added this pull request to the merge queue May 29, 2024
Merged via the queue into main with commit 6d76515 May 29, 2024
16 checks passed
@danroc danroc deleted the feature/add-get-balances branch May 29, 2024 09:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
team-accounts This should be handled by the Accounts Team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants