Skip to content

feat: add redirect grace period#258

Closed
dangelo352 wants to merge 1 commit into
Disciplr-Org:mainfrom
dangelo352:feature/redirect-grace-period-222
Closed

feat: add redirect grace period#258
dangelo352 wants to merge 1 commit into
Disciplr-Org:mainfrom
dangelo352:feature/redirect-grace-period-222

Conversation

@dangelo352

Copy link
Copy Markdown

Summary

  • add grace_period: u64 to ProductivityVault and create_vault
  • bound grace periods to MAX_VAULT_DURATION while preserving grace_period == 0 legacy redirect behavior
  • require redirect_funds to wait until now > end_timestamp + grace_period using checked addition
  • cover exact-boundary rejection, after-boundary success, oversized grace rejection, overflow rejection, and proptest storage/bounds
  • update README, backend docs, contract interface, and add docs/GRACE_PERIOD.md

Tests

  • cargo test --test lifecycle --test proptest_timestamps -- --nocapture
  • cargo test
  • cargo clippy --all-targets --all-features -- -D warnings
  • cargo fmt --check
  • git diff --check

Closes #222

@1nonlypiece

Copy link
Copy Markdown
Contributor

hey, good work, but this can't go in until you're assigned to #222 — that's our process to avoid two people on the same thing. closing for now; claim the issue first and re-raise. https://discord.gg/xvNAvMJf

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.

Allow creator-configurable grace period before redirect_funds in src/lib.rs

2 participants