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

fix: make expandatures stakes finalizable in UserHub menu #4238

Merged
merged 1 commit into from
Feb 5, 2025

Conversation

Nortsova
Copy link
Contributor

@Nortsova Nortsova commented Feb 3, 2025

Description

Expenditure stakes weren't shown as "Finalizable" in the User Hub menu.

Thanks to @jakubcolony for pair-programing this with me. 🙌

Note

Block-ingestor PR for this: JoinColony/block-ingestor#330

Testing

  1. Install the Voting Reputation extension
  2. Create Advanced Payments with the "Staking" decision method
  3. Verify that the stake appears in User Hub:
image 4. Press "Confirm details" -> Press "Fund" -> Select "Reputation" -> Fund payment 5. Fully support and fully oppose voting 6. Verify that there is a second stake in User Hub image
  1. Vote for "Support" and Reveal your vote
  2. Verify that second stake got "Finalizable" status
image
  1. Finalize it and check User Hub again. Status should be "Claimable"
  2. Release with "Reputation"
  3. Fully support and fully oppose voting
  4. Verify that there is a new Stake in the User Hub
image
  1. Vote for "Support" and Reveal your vote
  2. Verify that third stake got "Finalizable" status
image
  1. Finalize it and check User Hub again. Status should be "Claimable"
image
  1. Verify that dates for all stakes are the same

Resolves #4047

@Nortsova Nortsova requested a review from a team as a code owner February 3, 2025 20:30
@Nortsova Nortsova self-assigned this Feb 3, 2025
@Nortsova Nortsova linked an issue Feb 4, 2025 that may be closed by this pull request
iamsamgibbs
iamsamgibbs previously approved these changes Feb 4, 2025
Copy link
Contributor

@iamsamgibbs iamsamgibbs left a comment

Choose a reason for hiding this comment

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

Nice job working through this, seems like everything is working nicely! Checking the stake type rather than simply looking for an expenditureId feels a lot more robust.

Created a staking payment. Funded with reputation and got it to a finalizable stage:

Screenshot 2025-02-04 at 11 22 45

And progressed it to a claimable state:

Screenshot 2025-02-04 at 11 23 13

Then went through releasing with reputation:

Screenshot 2025-02-04 at 11 24 24

The only thing I noticed is that the release motion doesn't get the correct description here:

Screenshot 2025-02-04 at 11 24 53

It's kind of unrelated to the issue, but I'm not sure if you could fix it up here? Otherwise we can just make a new issue for it.

@Nortsova
Copy link
Contributor Author

Nortsova commented Feb 4, 2025

Thank you, @iamsamgibbs. Nice catch on the translation. I will handle it at #4143 🙌

@Nortsova Nortsova force-pushed the fix/4074-user-hub-stakes-finalizable branch from 4a19126 to d9b99d0 Compare February 4, 2025 14:59
Copy link
Contributor

@davecreaser davecreaser left a comment

Choose a reason for hiding this comment

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

Very nice @Nortsova changes all look good and it works as expected (thanks also for the super clear testing steps! 🚀)

Screenshot 2025-02-04 at 16 41 50 Screenshot 2025-02-04 at 16 42 32 Screenshot 2025-02-04 at 16 43 03 Screenshot 2025-02-04 at 16 43 27 Screenshot 2025-02-04 at 16 44 12 Screenshot 2025-02-04 at 16 44 55 Screenshot 2025-02-04 at 16 45 22

Copy link
Member

@rdig rdig left a comment

Choose a reason for hiding this comment

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

Nicely done here!

@Nortsova Nortsova merged commit 30994b9 into master Feb 5, 2025
2 checks passed
@Nortsova Nortsova deleted the fix/4074-user-hub-stakes-finalizable branch February 5, 2025 11:41
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.

[UserHub] Data isn't shown on Finilizible and Climable tabs of UserHub
4 participants