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

Transaction List - Claim update amounts incorrect #611

Closed
tzarebczan opened this issue Sep 24, 2017 · 3 comments
Closed

Transaction List - Claim update amounts incorrect #611

tzarebczan opened this issue Sep 24, 2017 · 3 comments
Assignees
Labels
area: ux type: bug Existing functionality is wrong or broken
Milestone

Comments

@tzarebczan
Copy link
Contributor

The Issue

Currently, an update claim transaction shows the bid amount as the amount of the transaction. Since an update tx takes the previous bid input, funds are moved from the previous claim to the new one, so they aren't really spent during a transaction where the bid is not changed. In this case, it would show 0 and only a fee.

There are also 2 other scenarios to take into account and calculate the transaction amount based on the original bid - a lower bid and higher bid. For a lower bid, you'd have a net positive amount on the update. For a higher bid, you would have a net negative amount.

Expected behaviour

Update tx amount should show net positive or negative bid amount

Actual behaviour

Update tx amount shows the bid amount

System Configuration

  • LBRY Daemon version:
  • LBRY App version:
  • LBRY Installation ID:
  • Operating system:

Anything Else

Screenshots

@tzarebczan tzarebczan added type: bug Existing functionality is wrong or broken area: ux labels Sep 24, 2017
@kauffj kauffj added this to the October 2 milestone Sep 26, 2017
@tzarebczan tzarebczan changed the title Transaction List - Claim updates amounts incorrect Transaction List - Claim update amounts incorrect Sep 27, 2017
@lyoshenka lyoshenka modified the milestones: October 2, October 16 Oct 3, 2017
@tzarebczan
Copy link
Contributor Author

Issue moved to lbryio/lbry #947 via ZenHub

@tzarebczan
Copy link
Contributor Author

@hackrush01, reopening this one app side for you based on the daemon implementation.

@tzarebczan tzarebczan reopened this Nov 7, 2017
@hackrush01
Copy link
Contributor

Fixed in #731.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: ux type: bug Existing functionality is wrong or broken
Projects
None yet
Development

No branches or pull requests

4 participants