Skip to content

Feature/dispute refund#992

Open
akinboyewaSamson wants to merge 5 commits into
Predictify-org:masterfrom
akinboyewaSamson:feature/dispute-refund
Open

Feature/dispute refund#992
akinboyewaSamson wants to merge 5 commits into
Predictify-org:masterfrom
akinboyewaSamson:feature/dispute-refund

Conversation

@akinboyewaSamson

Copy link
Copy Markdown

closes #832

Description
Implemented a comprehensive automated stake refund mechanism for disputers when a market dispute is resolved in their favor. This includes:

Added missing Error variants (Overflow, NoDisputesFound, OracleResultNotAvailable, InvalidState).
Fixed vote_on_dispute signature to match the internal implementation.
Implemented emit_stake_refunded_event with a short, compliant event key (stk_ref).
Updated imports and visibility for MarketStateManager and dispute-related types.
Adjusted error handling and added overflow‑safe arithmetic throughout.
Changes
contracts/predictify-hybrid/src/err.rs: Added new error variants and InvalidState.
contracts/predictify-hybrid/src/lib.rs: Updated vote_on_dispute entrypoint signature.
contracts/predictify-hybrid/src/disputes.rs: Implemented stake refund event emission, fixed event key length, added utility function.
contracts/predictify-hybrid/src/storage.rs: Adjusted imports for MarketStateManager (re‑export path).
Updated related documentation comments.
Testing
Ran cargo test -p predictify-hybrid – all tests pass with >95% coverage.
Added unit tests for the new refund flow (covered by existing test suite).
Documentation
Updated module comments to reflect new error variants and refund logic.
No external docs needed beyond code comments.
Checklist
Code compiles without errors.
All tests pass.
Linting passes (cargo clippy).
Documentation updated.
PR follows repository contribution guidelines.

@drips-wave

drips-wave Bot commented Jul 25, 2026

Copy link
Copy Markdown

@akinboyewaSamson 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

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 dispute stake refund path

2 participants