Skip to content

fix: Validate the reward asset address is set and non-default - #2310

Open
Muyideen-js wants to merge 2 commits into
EarnQuestOne:mainfrom
Muyideen-js:solver/issue-2289-63-a0
Open

Muyideen-js wants to merge 2 commits into
EarnQuestOne:mainfrom
Muyideen-js:solver/issue-2289-63-a0

Conversation

@Muyideen-js

Copy link
Copy Markdown

Summary

Implemented changes for issue #2289. The agent reached its turn budget after producing this draft; repository CI must validate the result.

Changed files

  • contracts/earn-quest/src/lib.rs
  • contracts/earn-quest/src/quest.rs
  • contracts/earn-quest/src/test_reward_asset.rs
  • contracts/earn-quest/src/validation.rs

Test plan

Run all repository CI checks and review the changed behavior.

This PR was created as a draft by the issue solver bot. It will remain a
draft until repository CI passes.

Closes #2289

@drips-wave

drips-wave Bot commented Aug 26, 2026

Copy link
Copy Markdown

@Muyideen-js 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

@Muyideen-js
Muyideen-js marked this pull request as ready for review August 26, 2026 09:59
@RUKAYAT-CODER

Copy link
Copy Markdown
Contributor

Well done on the job done so far!
Kindly fix workflow to pass

@RUKAYAT-CODER

Copy link
Copy Markdown
Contributor

Well done on the job done so far, kindly fix workflow to pass and resolve conflict

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.

Validate the reward asset address is set and non-default

3 participants