Skip to content

Wallet Import Does Not Show Full Transaction History #964

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

Closed
1 task done
atefnazmy opened this issue Mar 24, 2025 · 4 comments
Closed
1 task done

Wallet Import Does Not Show Full Transaction History #964

atefnazmy opened this issue Mar 24, 2025 · 4 comments
Assignees
Labels
type_bug Something isn't working
Milestone

Comments

@atefnazmy
Copy link

Is there an existing issue for this?

  • I have searched the existing issues

What happened?

When importing a wallet into another account, the transaction history does not display all past transactions. Instead, it only shows transactions from the current month.

Steps To Reproduce

1.Open the application.
2.Click on menu icon.
3.Click on "Wallet" in menu items.
4.Write your PIN code.
5..Click on "Transactions" in bottom center of the page.

Relevant screenshots/screen records

Screencast.From.2025-03-24.09-22-38.mp4

Relevant log output

n/a
@ramezsaeed ramezsaeed added this to 3.17 Mar 25, 2025
@ramezsaeed ramezsaeed moved this to Accepted in 3.17 Mar 25, 2025
@ramezsaeed ramezsaeed added this to the 4.3.x milestone Mar 25, 2025
@zaelgohary zaelgohary self-assigned this Apr 3, 2025
@zaelgohary zaelgohary added the type_bug Something isn't working label Apr 3, 2025
@zaelgohary zaelgohary moved this from Accepted to In Progress in 3.17 Apr 3, 2025
@zaelgohary
Copy link
Contributor

Work Completed:

Fixed getTransactions from dart client. Also edited listTransactions to use pagingToken instead of offset.

@zaelgohary
Copy link
Contributor

Currently working on getting memos in parallel instead of getting them one by one in each transaction.

@zaelgohary
Copy link
Contributor

Work Completed:

Refactored getTransactions to return a stream instead of list and returned memos in parallel. Also applied changes in app.

@zaelgohary zaelgohary moved this from In Progress to Pending Review in 3.17 Apr 9, 2025
@zaelgohary zaelgohary moved this from Pending Review to Pending Deployment in 3.17 Apr 9, 2025
@ramezsaeed ramezsaeed moved this from Pending Deployment to In Verification in 3.17 May 25, 2025
@khaledyoussef24
Copy link

The issue is no longer reproducible
Tried the same secrets and did not face any issues seeing the transaction history

Image

https://github.com/user-attachments/assets/4ea22496-5707-4ddf-b4d0-e2e2811d44bc
testlodge

@khaledyoussef24 khaledyoussef24 moved this from In Verification to Done in 3.17 May 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type_bug Something isn't working
Projects
Status: Done
Development

No branches or pull requests

4 participants