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

get_account_history return duplicate ops #1176

Closed
tianyekuo opened this issue Jul 25, 2018 · 1 comment
Closed

get_account_history return duplicate ops #1176

tianyekuo opened this issue Jul 25, 2018 · 1 comment
Labels
2e Ready for Testing Status indicating the solution is sufficiently developed to begin testing 3d Bug Classification indicating the existing implementation does not match the intention of the design 4b Normal Priority Priority indicating the moderate impact to system/user -OR- existing workaround is costly to perform 6 CLI Impact flag identifying the command line interface (CLI) wallet application bug cli
Milestone
201808 - Non-Cons...

Comments

@tianyekuo
Copy link

Bug Description
Call the cli_wallet api with the params:
{"id":"5665571840","jsonrpc":"2.0","method":"get_account_history","params":["alanguanwz9",1000]}
The return result contains duplicate id : 1.11.323175501, 1.11.320758270

"get_relative_account_history" api does not have this issue.

Screenshots (optional)
get_account_history
@abitmore abitmore added cli bug 2c Ready for Development Status indicating the Requirements are sufficent to begin designing a solution 3d Bug Classification indicating the existing implementation does not match the intention of the design 4b Normal Priority Priority indicating the moderate impact to system/user -OR- existing workaround is costly to perform 6 CLI Impact flag identifying the command line interface (CLI) wallet application labels Jul 25, 2018
@abitmore abitmore self-assigned this Jul 25, 2018
abitmore added a commit that referenced this issue Jul 25, 2018
@abitmore
Fix CLI get_account_history pagination issue #1176
1833a94
@abitmore abitmore mentioned this issue Jul 25, 2018
Merged
@abitmore abitmore added 2e Ready for Testing Status indicating the solution is sufficiently developed to begin testing and removed 2c Ready for Development Status indicating the Requirements are sufficent to begin designing a solution labels Jul 25, 2018
@abitmore abitmore removed their assignment Jul 25, 2018
@jmjatlanta jmjatlanta mentioned this issue Jul 25, 2018
Merged
abitmore added a commit that referenced this issue Jul 27, 2018
@abitmore
Merge pull request #1177 from bitshares/1176-cli-account-his
1608ce0 
Fix CLI get_account_history pagination issue #1176
@abitmore
Copy link
Member

Fixed with #1177.

@abitmore abitmore mentioned this issue Aug 4, 2018
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
2e Ready for Testing Status indicating the solution is sufficiently developed to begin testing 3d Bug Classification indicating the existing implementation does not match the intention of the design 4b Normal Priority Priority indicating the moderate impact to system/user -OR- existing workaround is costly to perform 6 CLI Impact flag identifying the command line interface (CLI) wallet application bug cli
Projects
None yet
Milestone
201808 - Non-Consensus-Changing Release
Development

No branches or pull requests
2 participants
@tianyekuo @abitmore