Add RPC-backed transaction history#69
Conversation
|
Simulator validation completed for this PR. Validation run:
Screenshots: |
There was a problem hiding this comment.
Pull request overview
This PR adds an in-app, RPC-backed transaction history experience for the wallet, replacing the previous “Records” behavior that navigated users out to an external explorer. It queries NEP-17 and NEP-11 transfers on demand via RPC and resolves token metadata in-memory for display.
Changes:
- Routed Wallet “Records” tap to a new in-app
TransactionHistoryPage(//wallet/transactions) and registered the Shell route. - Implemented
TransactionHistoryPage(UI + code-behind) to load NEP-17/NEP-11 transfers from RPC and render them as activity rows. - Added new localized string keys for empty/error/retry transaction history states across all
.resxfiles and updatedStrings.Designer.cs.
Reviewed changes
Copilot reviewed 20 out of 21 changed files in this pull request and generated 15 comments.
Show a summary per file
| File | Description |
|---|---|
| OneGateApp/Properties/Strings.resx | Adds transaction history empty/error/retry strings (neutral culture). |
| OneGateApp/Properties/Strings.de.resx | Adds transaction history strings for German locale. |
| OneGateApp/Properties/Strings.es.resx | Adds transaction history strings for Spanish locale. |
| OneGateApp/Properties/Strings.fr.resx | Adds transaction history strings for French locale. |
| OneGateApp/Properties/Strings.id.resx | Adds transaction history strings for Indonesian locale. |
| OneGateApp/Properties/Strings.it.resx | Adds transaction history strings for Italian locale. |
| OneGateApp/Properties/Strings.ja.resx | Adds transaction history strings for Japanese locale. |
| OneGateApp/Properties/Strings.ko.resx | Adds transaction history strings for Korean locale. |
| OneGateApp/Properties/Strings.nl.resx | Adds transaction history strings for Dutch locale. |
| OneGateApp/Properties/Strings.pt-BR.resx | Adds transaction history strings for Brazilian Portuguese locale. |
| OneGateApp/Properties/Strings.ru.resx | Adds transaction history strings for Russian locale. |
| OneGateApp/Properties/Strings.tr.resx | Adds transaction history strings for Turkish locale. |
| OneGateApp/Properties/Strings.vi.resx | Adds transaction history strings for Vietnamese locale. |
| OneGateApp/Properties/Strings.zh-Hans.resx | Adds transaction history strings for Simplified Chinese locale. |
| OneGateApp/Properties/Strings.zh-Hant.resx | Adds transaction history strings for Traditional Chinese locale. |
| OneGateApp/Properties/Strings.Designer.cs | Exposes new TransactionHistory* resource accessors. |
| OneGateApp/Pages/WalletPage.xaml | Changes “Records” tap to navigate in-app to transaction history route. |
| OneGateApp/AppShell.xaml.cs | Registers the new wallet/transactions Shell route. |
| OneGateApp/Pages/TransactionHistoryPage.xaml | Adds UI for list/empty/error/loading states with pull-to-refresh. |
| OneGateApp/Pages/TransactionHistoryPage.xaml.cs | Implements RPC transfer loading, token metadata caching, and row formatting. |
| OneGateApp/Models/TransactionHistoryItem.cs | Adds a simple UI model for transaction history rows. |
Files not reviewed (1)
- OneGateApp/Properties/Strings.Designer.cs: Generated file
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Review feedback fixes pushed in Validation after the update:
Screenshots: |
erikzhang
left a comment
There was a problem hiding this comment.
I tested it, but the historical transaction records were not displayed.
|
Addressed Erik's latest review requests in Changes:
Validation:
No screenshots were committed to the repository. |
|
It still doesn't work. |
|
Follow-up fix pushed in Root cause:
Fix:
Verification:
Screenshots were captured under |
|
Update: switched account history loading to use indexed account transactions before falling back to transfer-only RPC data. What changed:
Why:
Validation:
Screenshots captured outside the repo:
Note: build warnings are the existing SQLitePCLRaw NU1903 advisory warnings; no new warnings were introduced by this change. |
|
Follow-up update pushed in Change:
Verification:
Screenshots captured outside the repository:
Note: the existing SQLitePCLRaw |
|
Current head already addresses the open transaction-history feedback. Checked in this pass:
No additional commit was needed in this pass. |




Summary
Validation
Notes