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

Windows 11: Rewards: Rewards History displayed incorrectly. There is history that I have not won displayed #1317

Closed
Tas4tari opened this issue Dec 27, 2024 · 1 comment
Assignees
Labels
bug Something isn't working regression

Comments

@Tas4tari
Copy link

Tas4tari commented Dec 27, 2024

Describe the bug
As a user that has just imported a wallet that has a balance, I find that when I view Rewards, there is a list of rewards history that should not be there as I had only ever won 1 block and had the rest of the balance sent to me by someone.
The only block that I did mine successfully was on October 31- Block: #35744

To Reproduce
Steps to reproduce the behavior:

  1. Go to 'Tari Universe'
  2. Click on 'Settings' icon
  3. Scroll down to & click on 'Wallet`
  4. Scroll down to Seed words
  5. Click on the edit icon for Seed words
  6. Paste in the seed words of the wallet you would like to Import
  7. Import wallet
  8. When wallet is imported successfully, view wallet balance
  9. Click on the Rewards link within the Wallet balance section
  10. View Rewards history
  11. Observe

Expected behavior
User should only see the history of the blocks that they have won

Screenshots
Screenshot (7)
Screenshot (8)
Screenshot (9)
Screenshot (6)

Device:

  • Aspire A315-59
  • Windows 11, 24H2

Additional context
logs.zip

@Tas4tari Tas4tari changed the title Windows 11: Rewards: Rewards History does not align with wallet total. There is history should not be displayed as it does not belong to this wallet Windows 11: Rewards: Rewards History does not align with wallet total. There is history that should not be displayed as it does not belong to this wallet Dec 27, 2024
@Tas4tari Tas4tari changed the title Windows 11: Rewards: Rewards History does not align with wallet total. There is history that should not be displayed as it does not belong to this wallet Windows 11: Rewards: Rewards History displayed incorrectly. There is history that I have not won displayed Dec 27, 2024
@brianp brianp added regression bug Something isn't working labels Jan 8, 2025
@mmrrnn mmrrnn self-assigned this Jan 13, 2025
brianp added a commit that referenced this issue Jan 21, 2025
Fixes:
* #1346
* #1317
* partially #883

Description
--
Since tari-project/tari#6733 is merged, we can
now implement pagination for infinite scrolling. Wallets with extensive
transaction history can now open `latest wins` tab without performance
issues.

Changes
--
* Infinite scrolling in the "latest wins" tab
* Filter only `COINBASE_UNCONFIRMED` and `COINBASE_CONFIRMED`
transactions(blocks wins). Rename methods from `get_transaction_history`
to `get_coinbase_transactions` to reflect changes.
* `get_coinbase_transactions` handles pagination - we define
`continuation` and `limit`(can be ommitted to fetch all at once)
* We fetch already ordered transactions, we don't need to sort it again
* Refresh the data everytime when either we open "recent wins" tab or
new block is mined
* Split `Wallet` component into `WalletMarkup` and `History` to improve
performance(reduce rerenders)

Testing
--
Just play with the "recent wins" tab

---------

Co-authored-by: Brian Pearce <[email protected]>
@brianp
Copy link
Collaborator

brianp commented Jan 21, 2025

Should be fixed with #1343

@brianp brianp closed this as completed Jan 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working regression
Projects
None yet
Development

No branches or pull requests

4 participants
@brianp @mmrrnn @Tas4tari and others