Skip to content

fix(quest-engine): prevent duplicate explore quest payouts - #97

Closed
syed-ghufran-hassan wants to merge 5 commits into
learnault:mainfrom
syed-ghufran-hassan:feat/explore-verification-replay-guard
Closed

fix(quest-engine): prevent duplicate explore quest payouts#97
syed-ghufran-hassan wants to merge 5 commits into
learnault:mainfrom
syed-ghufran-hassan:feat/explore-verification-replay-guard

Conversation

@syed-ghufran-hassan

@syed-ghufran-hassan syed-ghufran-hassan commented Jul 26, 2026

Copy link
Copy Markdown
  • Add ExploreVerification DataKey to track (learner, quest_id)
  • Check verification marker before processing payout
  • Revert with clear panic if duplicate verification
  • Persist verification marker after successful payout
  • Add comprehensive tests for replay protection
  • Different learners on same quest remain independently payable

Closes #91

@Kaylahray

Copy link
Copy Markdown
Contributor

Hi @syed-ghufran-hassan I'm going to assign this issue to you because you applied. But I need you to create another branch or fix this one. I'll prefer you create a new branch so you don't overwrite the other changes I merged. @syed-ghufran-hassan

@syed-ghufran-hassan

Copy link
Copy Markdown
Author

Noted @Kaylahray I will create new branch

@syed-ghufran-hassan

syed-ghufran-hassan commented Jul 29, 2026

Copy link
Copy Markdown
Author

#102 - I have raised the PR @Kaylahray through separate branch as asked.

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.

[QuestEngine] Prevent duplicate Explore quest verification payouts

2 participants