Skip to content

Configurable retention#482

Draft
michielp1807 wants to merge 2 commits intomainfrom
configurable-retention
Draft

Configurable retention#482
michielp1807 wants to merge 2 commits intomainfrom
configurable-retention

Conversation

@michielp1807
Copy link
Collaborator

@michielp1807 michielp1807 commented Feb 4, 2026

TODO:

  • make project settings set retention period
  • make project creation choose a retention period
  • validate on server in create/update that retention period is reasonable based on subscription
  • add tests for retention period verification
  • make remove_expired_message_data use the project specific limit instead of '30 days'
  • what to do if an organization downgrades/ends their subscription? force lower retention period limit?
  • update existing tests for project-specific retention period?

Closes #424

@michielp1807 michielp1807 force-pushed the configurable-retention branch from 41214f1 to 4229166 Compare February 4, 2026 18:56
@codecov
Copy link

codecov bot commented Feb 4, 2026

Codecov Report

❌ Patch coverage is 84.78261% with 7 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/models/projects.rs 33.33% 4 Missing ⚠️
src/models/organization.rs 85.00% 3 Missing ⚠️

📢 Thoughts on this report? Let us know!

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.

Make retention configurable per project (with limits based on subscription)

1 participant