Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

fio.escrow API Endpoint #326

Merged
merged 2 commits into from
Dec 17, 2021
Merged

Conversation

tvl83
Copy link
Contributor

@tvl83 tvl83 commented Nov 30, 2021

endpoint to get listings by status with an optional parameter of the account name (actor). When filtering by actor, use limit 1000

Change Description

Consensus Changes

  • Consensus Changes

API Changes

  • API Changes

this PR adds an endpoint get_escrow_listings which takes 3 parameters. status, limit, and offset with an optional parameter actor. When the actor parameter is omitted, the limit and offset work fine, and only records of that given status are returned. If the actor parameter is provided it will only get the records with the requested status but also filtered on the given actor. The caveat is that the limit must be set to 1000 because of the additional filtering a lower limit will not work properly.

If there are 100 records and the 10th record matches the actor parameter but the limit is 1, no records are returned because it is not found within that 1 record.

Documentation Additions

  • Documentation Additions

…ount name (actor). When filtering by actor, use limit 1000
@ericbutz ericbutz marked this pull request as ready for review December 17, 2021 17:05
@ericbutz ericbutz merged commit 49a4b59 into fioprotocol:develop Dec 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants