Skip to content

max_active_bids_per_investor zero versus disabled semantics #730

@Baskarayelu

Description

@Baskarayelu

Description

Document and test meaning of zero limit (unlimited vs disabled) and admin configuration.

Requirements and context

  • Must be secure, tested, and documented
  • Should be efficient and easy to review
  • Align with existing Soroban contract patterns in quicklendx-contracts

Suggested execution

  • Fork the repo and create a branch
  • git checkout -b feature/max-active-bids-investor-semantics
  • Implement changes
  • Write contract: src/bid.rs
  • Write comprehensive tests: src/test_bid.rs, src/test_limit.rs
  • Add documentation: docs/contracts/limits.md
  • Include NatSpec-style comments (Rust doc comments on public items)
  • Validate security assumptions

Test and commit

  • Run tests
  • Cover edge cases
  • Include test output and security notes

Example commit message

feat: clarify max active bids per investor semantics

Guidelines

  • Minimum 95 percent test coverage
  • Clear documentation
  • Timeframe: 96 hours

Metadata

Metadata

Assignees

Labels

Stellar WaveIssues in the Stellar wave programbiddingDesign / UX backlog: biddingpriority-lowAuto label for contract issues: priority-lowsmart-contractsAuto label for contract issues: smart-contractstestingAuto label for contract issues: testingvalidationAuto label for contract issues: validation

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions