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

Set wallet representative with RPC receive on unopened accounts #2419

Conversation

guilhermelawless
Copy link
Contributor

@guilhermelawless guilhermelawless commented Nov 27, 2019

Closes #2418

Will update documentation, should state that the wallet representative will be used from V21, and the own account is used in V20 and before, for unopened accounts.

Adds a suite of tests for RPC receive.

@guilhermelawless guilhermelawless added bug documentation This item indicates the need for or supplies updated or expanded documentation unit test Related to a new, changed or fixed unit test labels Nov 27, 2019
@guilhermelawless guilhermelawless added this to the V21.0 milestone Nov 27, 2019
@guilhermelawless guilhermelawless self-assigned this Nov 27, 2019
@guilhermelawless guilhermelawless removed the request for review from wezrule December 4, 2019 18:44
@guilhermelawless guilhermelawless merged commit 14876af into nanocurrency:develop Dec 4, 2019
guilhermelawless added a commit to guilhermelawless/nano-node that referenced this pull request Dec 4, 2019
Changes from nanocurrency#2388 were not factored into nanocurrency#2419 since there were new RPC tests
guilhermelawless added a commit that referenced this pull request Dec 5, 2019
Changes from #2388 were not factored into #2419 since there were new RPC tests
@zhyatt zhyatt added the qt wallet Item related to the qt wallet label Apr 22, 2020
@guilhermelawless guilhermelawless deleted the rpc/receive-wrong-representative branch August 3, 2020 19:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug documentation This item indicates the need for or supplies updated or expanded documentation qt wallet Item related to the qt wallet unit test Related to a new, changed or fixed unit test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Wallet RPC action "receive" sets the wrong representative/ignores the wallet representative
4 participants