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.

Sign up for GitHub

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

Jump to bottom

RPC accounts_receivable applies count restriction before sorting #3801

Open
dsiganos opened this issue Apr 25, 2022 · 1 comment
Open

RPC accounts_receivable applies count restriction before sorting #3801

dsiganos opened this issue Apr 25, 2022 · 1 comment
Labels
documentation This item indicates the need for or supplies updated or expanded documentation rpc Changes related to Remote Procedure Calls
Milestone
Backlog

Comments

@dsiganos
Copy link
Contributor

RPC accounts_pending applies count before sorting.
If both count and sorting are set, then it should get all results, sort them and then apply count.
The documentation also needs to be fixed to reflect that change.
@dsiganos dsiganos added the rpc Changes related to Remote Procedure Calls label Apr 25, 2022
@filipesmedeiros
Copy link

Why does this matter? Is the count supposed to change after sorting?

@JerzyStanislawski JerzyStanislawski mentioned this issue Jun 27, 2022
Closed
@thsfs thsfs added the documentation This item indicates the need for or supplies updated or expanded documentation label Jun 27, 2022
@qwahzi qwahzi added this to the Backlog milestone Jan 4, 2023
@gr0vity-dev gr0vity-dev mentioned this issue Jan 10, 2023
Open
JerzyStanislawski added a commit to JerzyStanislawski/nano-node that referenced this issue Jan 24, 2023
@JerzyStanislawski
Issue nanocurrency#3801 fix receivable rpc method using ranges - draft
f077812
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
documentation This item indicates the need for or supplies updated or expanded documentation rpc Changes related to Remote Procedure Calls
Projects
None yet
Milestone
Backlog
Development

No branches or pull requests
4 participants
@dsiganos @qwahzi @filipesmedeiros @thsfs