Skip to content

PR for issue #387#456

Merged
RUKAYAT-CODER merged 4 commits intoEarnQuestOne:mainfrom
Levi-Ojukwu:main
Apr 29, 2026
Merged

PR for issue #387#456
RUKAYAT-CODER merged 4 commits intoEarnQuestOne:mainfrom
Levi-Ojukwu:main

Conversation

@Levi-Ojukwu
Copy link
Copy Markdown
Contributor

Linked Issue

Closes #387

Description

Implemented support for partial claims in the payout logic, allowing users to claim rewards in flexible amounts instead of requiring full withdrawals.

What changed?

  • Added logic to support partial claim amounts in payout.rs
  • Updated payout calculations to handle remaining balances after partial claims
  • Added validation to prevent over-claiming
  • Wrote tests covering partial and full claim scenarios
  • Updated inline documentation for clarity

Why was it changed?

  • Existing implementation only supported full claims, limiting flexibility
  • Improves user experience by allowing incremental withdrawals

How was it implemented?

  • Modified claim function to accept and validate partial amounts
  • Ensured contract state updates correctly after each claim
  • Added safeguards to prevent claiming more than available balance
  • Covered edge cases (e.g., zero claim, full claim after partials)

@drips-wave
Copy link
Copy Markdown

drips-wave Bot commented Apr 28, 2026

@Levi-Ojukwu Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@RUKAYAT-CODER
Copy link
Copy Markdown
Contributor

kindly resolve conflict

@Levi-Ojukwu
Copy link
Copy Markdown
Contributor Author

Conflict has been resolved. Please merge.

The CI checks are not passing. I've tried everything.

@RUKAYAT-CODER RUKAYAT-CODER merged commit 4ef14eb into EarnQuestOne:main Apr 29, 2026
0 of 7 checks passed
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.

Partial Claims Support

2 participants