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

feat: shielded rewards intergration #1378

Merged
merged 6 commits into from
Jan 31, 2025
Merged

Conversation

mateuszjasiuk
Copy link
Collaborator

@mateuszjasiuk mateuszjasiuk commented Dec 4, 2024

Resolves #1371

This integrates the shielded rewards.
The downside is that is seems that the endpoint in namada rpc returns some weird values, so testing might be a bit tricky.
How it works for me atm is that if I shield a lot of nam and wait one masp epoch I will see some rewards for one masp epoch, but then it will go back to 0.
You can shield more and repeat the process

Not sure if campfire has nam rewards enabled :/
Also added a function which can simulate rewards for this:
image

@mateuszjasiuk mateuszjasiuk force-pushed the feat/shielded-rewards branch 2 times, most recently from fb410c4 to 4794a30 Compare December 11, 2024 12:17
@mateuszjasiuk mateuszjasiuk marked this pull request as draft December 11, 2024 12:20
@mateuszjasiuk
Copy link
Collaborator Author

mateuszjasiuk commented Dec 11, 2024

Moved to draft as Marco is changing some stuff in namada.

@mateuszjasiuk mateuszjasiuk marked this pull request as ready for review January 29, 2025 10:38
Copy link
Collaborator

@jurevans jurevans left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code looks good! Tested locally, shielded rewards look great!

@mateuszjasiuk mateuszjasiuk merged commit 622044d into main Jan 31, 2025
7 checks passed
@mateuszjasiuk mateuszjasiuk deleted the feat/shielded-rewards branch January 31, 2025 14:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add endpoint for shielded rewards
2 participants