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

Prevent exception when querying account history #1494

Merged
merged 3 commits into from Dec 31, 2018

Conversation

Projects
3 participants
@jmjatlanta
Copy link
Contributor

commented Dec 25, 2018

Fixes #1490

When calling get_account_history_operations using a node with limited history, an exception can be thrown that prevents the user from getting any information.

This fix provides the history up to the limit of the connected node.

jmjatlanta added some commits Dec 25, 2018

Show resolved Hide resolved libraries/app/api.cpp Outdated

@jmjatlanta jmjatlanta merged commit 143ee6f into develop Dec 31, 2018

2 of 3 checks passed

ci/dockercloud Your tests are pending in Docker Cloud
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@jmjatlanta jmjatlanta deleted the jmj_1490 branch Dec 31, 2018

@abitmore abitmore added this to In progress in Feature Release (201902) via automation Jan 1, 2019

@abitmore abitmore added this to the 201901 - Feature Release milestone Jan 1, 2019

@abitmore abitmore moved this from In progress to Done in Feature Release (201902) Jan 1, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.