Skip to content

feat: add funding target, deadline, deposit limits to vault response#556

Merged
Jayy4rl merged 2 commits into
StellarYield:mainfrom
gerani1:feat/issues-518-519-vault-funding-deposit-limits
May 30, 2026
Merged

feat: add funding target, deadline, deposit limits to vault response#556
Jayy4rl merged 2 commits into
StellarYield:mainfrom
gerani1:feat/issues-518-519-vault-funding-deposit-limits

Conversation

@gerani1
Copy link
Copy Markdown
Contributor

@gerani1 gerani1 commented May 29, 2026

Closes #518,
Closes #519

  • Add migration 009 with funding_target, funding_deadline, min_deposit, max_deposit_per_user columns to vaults table
  • Extend Vault type and VaultRow with new nullable fields
  • Compute fundingProgress (0–100, capped) from totalAssets/fundingTarget
  • Update all SELECT queries and upsertVault to include new columns
  • Parse new fields from vault_created XDR event in indexer

Pull Request Checklist

General

  • PR title follows conventional commit format (type(scope): description)
  • Branch name follows issue-<number>-<description> format
  • Linked to relevant issue(s) with Closes #<number> or Fixes #<number>
  • Description clearly explains what changes were made and why
  • Breaking changes documented (if applicable)

Code Quality

  • Code follows project style guidelines (make fmt passes)
  • No clippy warnings (make lint passes)
  • No new panic! statements (use panic_with_error! instead)
  • Public functions have proper documentation
  • Complex logic is explained with comments

Testing

  • All existing tests still pass (make test passes)
  • New functionality is covered by tests
  • Test coverage meets minimum requirements (90%+ for new features)
  • Tests follow naming conventions (test_<functionality>_scenario)
  • Edge cases and error conditions are tested

Security & Events

  • Access control properly implemented and tested
  • All state changes emit appropriate events
  • Input validation is comprehensive
  • Reentrancy protection where needed
  • CEI pattern (Checks-Effects-Interactions) followed

Documentation

  • README updated (if applicable)
  • API documentation updated (if applicable)
  • Architecture docs updated (if applicable)
  • Examples added or updated (if applicable)

Description

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Refactoring (no functional changes)
  • Performance improvement
  • Security improvement

Testing

Security Considerations

Screenshots / Diagrams

Additional Context

Reviewer Focus Areas


By submitting this PR, I confirm that:

  • I have read and followed the contribution guidelines
  • My code follows the project's style standards
  • I have performed a self-review of my code
  • I have added tests that prove my fix is effective or that my feature works
  • I have tested my changes thoroughly
  • I understand that this PR may be closed if I don't respond to feedback in a timely manner

Closes StellarYield#518, Closes StellarYield#519

- Add migration 009 with funding_target, funding_deadline, min_deposit,
  max_deposit_per_user columns to vaults table
- Extend Vault type and VaultRow with new nullable fields
- Compute fundingProgress (0–100, capped) from totalAssets/fundingTarget
- Update all SELECT queries and upsertVault to include new columns
- Parse new fields from vault_created XDR event in indexer

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@drips-wave
Copy link
Copy Markdown

drips-wave Bot commented May 29, 2026

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

@Jayy4rl
Copy link
Copy Markdown
Contributor

Jayy4rl commented May 30, 2026

@gerani1 check and resolve conflicts

@Jayy4rl Jayy4rl merged commit 0e69c6c into StellarYield:main May 30, 2026
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 funding_target and funding_deadline to vault response Add min_deposit and max_deposit_per_user to vault response

3 participants