-
Couldn't load subscription status.
- Fork 112
Repair the StakeHistory sysvar fetcher
#375
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
Repair the StakeHistory sysvar fetcher
#375
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
🦋 Changeset detectedLatest commit: 5749656 The changes in this PR will be included in the next version bump. This PR includes changesets to release 39 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
This stack of pull requests is managed by Graphite. Learn more about stacking. |
e7e76fe to
35cf47d
Compare
6fef961 to
20e498c
Compare
BundleMonFiles updated (3)
Unchanged files (124)
Total files change +306B +0.09% Final result: ✅ View report in BundleMon website ➡️ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah yep, whoops, looks like the epoch number is in there. Thanks!
https://github.com/solana-program/stake/blob/bcec951fda5f2a30b1f4a058706d2e9ed23a8429/interface/src/stake_history.rs#L66
| deactivating: Lamports; | ||
| effective: Lamports; | ||
| epoch: Epoch; | ||
| stakeHistory: Readonly<{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
up-to-you:
| stakeHistory: Readonly<{ | |
| stakeHistoryEntry: Readonly<{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just the nomenclature used in the Stake Interface, for consistency. Doesn't really matter, though.
https://github.com/solana-program/stake/blob/bcec951fda5f2a30b1f4a058706d2e9ed23a8429/interface/src/stake_history.rs#L17
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I went with that which is spit out by the jsonParsed parser of the RPC.
¯_(°ペ)_/¯
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
R+ with Joe's comment!
35cf47d to
d8a27fa
Compare
6cbe362 to
938dd25
Compare
d8a27fa to
498073a
Compare
Merge activity
|
498073a to
f5b85fb
Compare
938dd25 to
5749656
Compare
|
Because there has been no activity on this PR for 14 days since it was merged, it has been automatically locked. Please open a new issue if it requires a follow up. |
Problem
The data from this decoder is malformed.
Test Plan
Compare that to: