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

[Model][Backport] Remove mapWallet not needed call + stop treating coinbase differently #1073

Open
wants to merge 1 commit into
base: master
from

Conversation

@furszy
Copy link
Collaborator

furszy commented Oct 30, 2019

Remove mapWallet not needed call + stop treating coinbases differently.

Coming from bitcoin/bitcoin@608bbcc.

src/qt/transactionrecord.cpp Outdated Show resolved Hide resolved
@furszy furszy force-pushed the furszy:2019_txModel_showTransaction_always branch from 6c95fae to 707a827 Nov 4, 2019
@furszy furszy requested review from Warrows and random-zebra Nov 4, 2019
Copy link
Collaborator

random-zebra left a comment

utACK 707a827

@furszy furszy requested a review from Fuzzbawls Nov 6, 2019
@random-zebra

This comment has been minimized.

Copy link
Collaborator

random-zebra commented Nov 10, 2019

needs rebase

@random-zebra random-zebra added this to the 4.0.0 milestone Nov 10, 2019
…se outputs differently (coming from 608bbcc).

[Model] TransactionRecord, showTransaction method removed.
@furszy furszy force-pushed the furszy:2019_txModel_showTransaction_always branch from 707a827 to 856ba66 Nov 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.