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

get to_id from to_account instead of get_account_id() #1242

Merged
merged 1 commit into from Aug 9, 2018

Conversation

Projects
5 participants
@RichardWeiYang

RichardWeiYang commented Aug 9, 2018

This would reduce one rpc call since get_account_id() retrieve data from
remote.

get to_id from to_account instead of get_account_id()
This would reduce one rpc call since get_account_id() retrieve data from
remote.
@pmconrad

Looks good, thanks.

@oxarbitrage oxarbitrage self-requested a review Aug 9, 2018

@oxarbitrage

nice one

@oxarbitrage oxarbitrage merged commit 7fbbbcf into bitshares:develop Aug 9, 2018

2 checks passed

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

@abitmore abitmore added this to In Development in Feature Release (201808) via automation Aug 15, 2018

@abitmore abitmore removed this from In Development in Feature Release (201808) Aug 15, 2018

@abitmore abitmore added this to In progress in Feature release (201810) via automation Aug 15, 2018

@abitmore abitmore moved this from In progress to Done in Feature release (201810) Aug 15, 2018

@ryanRfox

This comment has been minimized.

Member

ryanRfox commented Aug 20, 2018

@RichardWeiYang Thank you for the contributions to BitShares development. I hope to connect with you directly about the Community Claims compensation available through the BitShares Core Team Worker Proposal. You may reach me on Telegram or WeChat: ryanRfox

Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment