Skip to content

C-1: Audit market, factory and treasury contracts after the batch PR merges #1511

Closed

Description

@Ehonrie

Description: PRs #1500, #1501 and #1502 each rewrote overlapping parts of contracts/market, contracts/market_factory, contracts/treasury and contracts/shared. Their merge conflicts were large, and Contracts CI fails on the merged result, so the combined behaviour has not been verified.

Where: contracts/market/src/lib.rs, contracts/market_factory/src/lib.rs, contracts/treasury/src/lib.rs, contracts/shared/src/

Acceptance Criteria:

  • Merged contracts compile and all tests pass
  • Event emission uses the shared market_id_to_u64 consistently across contracts
  • Factory to treasury integration and deposit/payout validation reviewed for lost or duplicated checks
  • Error code snapshots updated and reviewed

Activity

  1. optimist82 commented on Oct 21, 2026

    @optimist82

    @optimist82 has applied to work on this issue as part of the Stellar Wave Program's 10th wave.

    Hi, I have carefully reviewed the requirements and I understand the issue. I can implement a clean and efficient solution that meets the acceptance criteria. Please assign me for this role. Thank you.

    ℹ️ Repo Maintainers: To accept this application, review their application or assign @optimist82 to this issue.

  2. stacyelliott492-star commented on Oct 21, 2026

    @stacyelliott492-star

    @stacyelliott492-star has applied to work on this issue as part of the Stellar Wave Program's 10th wave.

    I'd like to work on this issue.I've reviewed the problem description and the affected files, and I'm confident I can implement a clean fix that aligns with the project's current patterns.Please assign this issue to me so I can get started on a PR.Thank you!

    ℹ️ Repo Maintainers: To accept this application, review their application or assign @stacyelliott492-star to this issue.

  3. Akinloluwa20 commented on Oct 21, 2026

    @Akinloluwa20

    @Akinloluwa20 has applied to work on this issue as part of the Stellar Wave Program's 10th wave.

    I would love to work on this issue

    ℹ️ Repo Maintainers: To accept this application, review their application or assign @Akinloluwa20 to this issue.

  4. Ranjeet2063 commented on Oct 21, 2026

    @Ranjeet2063

    @Ranjeet2063 has applied to work on this issue as part of the Stellar Wave Program's 10th wave.

    Ready to ship immediately! Focus: Smart Contract & Systems Architecture (Rust / Soroban / Web3) for "C-1: Audit market, factory and treasury contracts ". Plan: Implement idiomatic Rust logic with strict authorization checks, gas-efficient storage patterns, and defensive error handling. Guaranteed 100% green CI & test coverage. Zero hand-holding. SLA: clean PR in 1-2h. Track record: 166+ merged PRs in Web3 (OSS score 83, Top 2%).

    ℹ️ Repo Maintainers: To accept this application, review their application or assign @Ranjeet2063 to this issue.

  5. mysteriousskater commented on Oct 21, 2026

    @mysteriousskater

    @mysteriousskater has applied to work on this issue as part of the Stellar Wave Program's 10th wave.

    Hello I’ll like to fix this issue

    ℹ️ Repo Maintainers: To accept this application, review their application or assign @mysteriousskater to this issue.

  6. iheomadev commented on Oct 21, 2026

    @iheomadev
    Contributor

    @iheomadev has applied to work on this issue as part of the Stellar Wave Program's 10th wave.

    Hello Maintainer, I would appreciate it if you could review my application and assign the issues to me. I am ready to get started and will ensure timely delivery.

    ℹ️ Repo Maintainers: To accept this application, review their application or assign @iheomadev to this issue.

  7. drips-wave commented on Oct 21, 2026

    @drips-wave

    Congratulations, @iheomadev! 🎉 Your application was accepted by the repo's maintainers, and the issue is due on October 22, 2026.

    🧑‍💻 @iheomadev: Please resolve the issue such that the repo's maintainers have enough time to review your contribution before the due date. You'll earn Points for completing the issue on-time, which will make you eligible for a share of the Stellar Wave Program's reward pool.

    Warning

    When opening a PR, please link it to this issue to ensure it gets tracked accurately. Points are awarded when this issue is marked as completed by the maintainer.

    🤠 Repo maintainers: Please keep an eye on the contributor's progress and review their work before the due date. You can manage this issue, including adjusting its complexity and points, here.

    🌊 Happy Wave 🌊

  8. drips-wave commented on Oct 22, 2026

    @drips-wave

    This issue has been completed by @iheomadev as part of the Stellar Wave Program's 10th Wave 🥳

    😎 @iheomadev: You earned 200 Points for completing this issue! Once the Wave's rewards are finalized, you'll be eligible for a percentage of the Wave's reward pool based on the percentage of total points you've earned. Learn more here. You can also Leave a review to share your experience working on this issue.

    🧑‍💻 Repo maintainers: How'd the contributor do? Leave a review to share your experience working with them.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions