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: Staking/RewardPool contracts #410

Merged
merged 4 commits into from
Apr 24, 2023
Merged

Fix: Staking/RewardPool contracts #410

merged 4 commits into from
Apr 24, 2023

Conversation

leric7
Copy link
Collaborator

@leric7 leric7 commented Apr 17, 2023

Description

Fixed Staking/RewardPool contracts.

Summary of changes

Some of the validations were wrong/missing.

How test the changes

Related issues

Keywords for linking issues

Closes #389
Closes #390
Closes #391
Closes #394

Operational checklist

  • All new functionality is covered by tests
  • Any related documentation has been changed or added

@vercel
Copy link

vercel bot commented Apr 17, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
escrow-dashboard ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 23, 2023 8:11pm
faucet-server ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 23, 2023 8:11pm
fortune-exchange-oracle-server ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 23, 2023 8:11pm
5 Ignored Deployments
Name Status Preview Comments Updated (UTC)
fortune-exchange-oracle ⬜️ Ignored (Inspect) Apr 23, 2023 8:11pm
fortune-job-launcher-client ⬜️ Ignored (Inspect) Apr 23, 2023 8:11pm
fortune-job-launcher-server ⬜️ Ignored (Inspect) Apr 23, 2023 8:11pm
fortune-recording-oracle ⬜️ Ignored (Inspect) Apr 23, 2023 8:11pm
fortune-reputation-oracle ⬜️ Ignored (Inspect) Apr 23, 2023 8:11pm

Copy link
Collaborator

@portuu3 portuu3 left a comment

Choose a reason for hiding this comment

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

Update tests please

@leric7 leric7 requested a review from portuu3 April 20, 2023 22:02
@leric7
Copy link
Collaborator Author

leric7 commented Apr 20, 2023

@portuu3 , added test cases to cover these changes. I think we need to setup coverage test in the near future.

@portuu3 portuu3 merged commit bb64fc3 into develop Apr 24, 2023
@portuu3 portuu3 deleted the fix/core-staking branch April 24, 2023 08:20
m00n620 pushed a commit that referenced this pull request May 4, 2023
* fix core staking / reward pool contracts

* add test cases
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.

2 participants