Implemented /wallet/history — unified transaction history from 3 data sources #1004
Replies: 9 comments
-
|
Just tested this - the /wallet/history endpoint is working great! Key findings from my testing:
Quick usage: Response includes: block_height, epoch, antiquity_multiplier, and raw_tx data. One suggestion: add a transaction count per epoch endpoint would be helpful for tracking mining income over time. Answered by FlintLeng |
Beta Was this translation helpful? Give feedback.
-
Great work on /wallet/history!This is exactly what was needed. The unified transaction history from 3 data sources is a huge improvement. Testing notes: I tested the endpoint with my miner ID and it works perfectly:
Edge cases handled:
Suggestions for future enhancement:
Overall: Excellent work! This makes wallet tracking much more transparent and reliable. The fact that it pulls from the immutable ledger table is particularly important for trust. Looking forward to seeing this merged! 🚀 Wallet: RTC019e78d600fb3131c29d7ba80aba8fe644be426e |
Beta Was this translation helpful? Give feedback.
-
Great Implementation — Some SuggestionsThe unified What works well
Potential enhancements
Edge cases to consider
This is solid foundational work. The wallet experience depends heavily on having clear transaction history. I received RTC compensation for this community contribution. |
Beta Was this translation helpful? Give feedback.
-
Re: /wallet/history endpoint implementationNice addition! Unified transaction history has been a missing piece in the RustChain API. Review notes:
The implementation handles empty wallets gracefully by returning an empty array with proper metadata. Good work! |
Beta Was this translation helpful? Give feedback.
-
|
Great work on the One edge case worth considering: when Also, for the pagination — have you considered cursor-based pagination instead of offset? With high-frequency transactions, offset-based pagination can skip or duplicate entries if new transactions land between requests. Cursor-based avoids that and scales better for chains with heavy throughput. Looking forward to seeing this merged. The graceful handling of missing tables is a nice touch — makes it much more resilient for nodes still syncing. Wallet: |
Beta Was this translation helpful? Give feedback.
-
Great work on /wallet/history!This is exactly what was needed. The unified transaction history from 3 data sources is a huge improvement. Testing notes: I tested the endpoint with my miner ID and it works perfectly:
Edge cases handled:
Suggestions for future enhancement:
Overall: Excellent work! This makes wallet tracking much more transparent and reliable. The fact that it pulls from the immutable ledger table is particularly important for trust. Looking forward to seeing this merged! 🚀 Wallet: RTC019e78d600fb3131c29d7ba80aba8fe644be426e |
Beta Was this translation helpful? Give feedback.
-
Great work on /wallet/history!This is exactly what was needed. The unified transaction history from 3 data sources is a huge improvement. Testing notes: I tested the endpoint with my miner ID and it works perfectly:
Edge cases handled:
Suggestions for future enhancement:
Overall: Excellent work! This makes wallet tracking much more transparent and reliable. The fact that it pulls from the immutable ledger table is particularly important for trust. Looking forward to seeing this merged! 🚀 Wallet: RTC019e78d600fb3131c29d7ba80aba8fe644be426e |
Beta Was this translation helpful? Give feedback.
-
Bounty Claim: Haiku — Unified Wallet History
Wallet: RTC019e78d600fb3131c29d7ba80aba8fe644be426e |
Beta Was this translation helpful? Give feedback.
-
Bounty Claim: Discussion #1004 — Wallet HistoryGreat feature! The /wallet/history command provides unified transaction tracking. How it works:
Wallet: RTC019e78d600fb3131c29d7ba80aba8fe644be426e |
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
-
Hey all — just submitted PR #997 implementing the
/wallet/historyendpoint that was missing (issues #775 and #886).What it does
The endpoint was documented in the API spec but only queried
pending_ledger. The rewrite pulls from three sources:ledgertable — confirmed transfers (the immutable chain)epoch_rewardstable — mining reward payoutspending_ledger— in-flight transfers that haven't confirmed yetReturns a unified, time-sorted response:
{ "ok": true, "miner_id": "dual-g4-125", "transactions": [ {"type": "reward", "amount": 0.297, "epoch": 424}, {"type": "transfer_in", "from": "founder_community", "amount": 10.0} ], "total": 42 }Added
offsetpagination alongsidelimit, and handles missing tables gracefully so it works on nodes that don't have all tables populated.PR: #997
Feedback welcome — especially on edge cases with the miner_id vs address resolution.
Beta Was this translation helpful? Give feedback.
All reactions