Skip to content
This repository has been archived by the owner on Jan 13, 2025. It is now read-only.

Add getTokenLargestAccounts endpoint #11322

Merged
merged 1 commit into from
Aug 2, 2020
Merged

Add getTokenLargestAccounts endpoint #11322

merged 1 commit into from
Aug 2, 2020

Conversation

CriesofCarrots
Copy link
Contributor

@CriesofCarrots CriesofCarrots commented Jul 31, 2020 •
edited
Loading

Problem

As per #11231 (review), the token equivalent of getLargestAccounts would be helpful for #11241

Summary of Changes

Add getLargestTokenAccounts rpc endpoint

Fixes #11260

CC @jstarry

@CriesofCarrots CriesofCarrots requested a review from mvines July 31, 2020 22:26
@codecov
Copy link

codecov bot commented Aug 1, 2020

Codecov Report

Merging #11322 into master will decrease coverage by 0.0%.
The diff coverage is 90.0%.

@@            Coverage Diff            @@
##           master   #11322     +/-   ##
=========================================
- Coverage    82.5%    82.5%   -0.1%     
=========================================
  Files         317      317             
  Lines       75186    75255     +69     
=========================================
+ Hits        62081    62118     +37     
- Misses      13105    13137     +32

@CriesofCarrots CriesofCarrots merged commit d1b2e6c into solana-labs:master Aug 2, 2020
mergify bot pushed a commit that referenced this pull request Aug 2, 2020
@CriesofCarrots
Add getTokenLargestAccounts endpoint (#11322)
fd02f37 
(cherry picked from commit d1b2e6c)
@mergify mergify bot mentioned this pull request Aug 2, 2020
Merged
mergify bot added a commit that referenced this pull request Aug 2, 2020
@mergify @CriesofCarrots
Add getTokenLargestAccounts endpoint (bp #11322) (#11338)
c769bcc 
* Add getTokenLargestAccounts endpoint (#11322)


(cherry picked from commit d1b2e6c)

* Rebase on v1.2

Co-authored-by: Tyera Eulberg <teulberg@gmail.com>
Co-authored-by: Tyera Eulberg <tyera@solana.com>
@CriesofCarrots CriesofCarrots deleted the token-largest-accts branch September 1, 2020 19:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@mvines mvines mvines approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Need the token equivalent of getLargestAccounts
2 participants
@CriesofCarrots @mvines