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

Change display logic for transaction table and scrape transactions #430

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Change display logic for transaction table and scrape transactions #430

wants to merge 2 commits into from

Conversation

IngCr3at1on
Copy link
Contributor

All scrape transactions display the address that the reward went to.

"External scrape" type is set on transactions where the staking address
is not found in the receiving wallet.

"Local scrape" type is set on transactions where the staking address
and reward/scrape address are both found in the same wallet.

"Scraped to external" type is set on transactions where the reward
address was not found in the same wallet that did the stake
In the case of such transactions the reward coins are never present in
the staking wallet and therefore are always displayed in brackets.

All scrape transactions display the address that the reward went to.

"External scrape" type is set on transactions where the staking address
is not found in the receiving wallet.

"Local scrape" type is set on transactions where the staking address
and reward/scrape address are both found in the same wallet.

"Scraped to external" type is set on transactions where the reward
address was not found in the same wallet that did the stake
In the case of such transactions the reward coins are never present in
the staking wallet and therefore are always displayed in brackets.
@IngCr3at1on
Copy link
Contributor Author

selection_190
selection_191

This addresses #210

(3 hours 10 minutes)

@IngCr3at1on
Copy link
Contributor Author

Added a commit to reduce some repeat code (15 minutes)

@IngCr3at1on
Copy link
Contributor Author

@MitchellCash and @CeForce it would be good to get some testing on this so we can merge it as well.

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

Successfully merging this pull request may close these issues.

1 participant