Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Fetch all from storage map #147

Closed
wants to merge 2 commits into from
Closed

Fetch all from storage map #147

wants to merge 2 commits into from

Conversation

ascjones
Copy link
Contributor

@ascjones ascjones commented Aug 3, 2020

WIP Closes #144.

Mistakenly pushed the first part of this in 663934c.

todo

  • Create an Iterator to perform paging requests
  • Add client config to override default page size
  • Add method for creating the iterator in proc_macro e.g. fn accounts_iter()

@dvc94ch
Copy link
Contributor

dvc94ch commented Aug 4, 2020

Is this up for grabs? We need this feature

@dvc94ch dvc94ch mentioned this pull request Aug 4, 2020
Merged
@ascjones
Copy link
Contributor Author

ascjones commented Aug 4, 2020

Closing in favour of #148

@ascjones ascjones closed this Aug 4, 2020
@ascjones ascjones deleted the aj-storage-map-fetch-all branch September 1, 2020 10:32
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Need 'fetch all the values in a map'
2 participants
@ascjones @dvc94ch