Skip to content

Conversation

yemkareems
Copy link
Contributor

☑️ Resolves

  • Fix #…

🖼️ Screenshots

@yemkareems yemkareems requested a review from artonge October 13, 2025 06:42
@yemkareems yemkareems self-assigned this Oct 13, 2025
Copy link
Contributor

@artonge artonge left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think, instead of adding another paragraph, we need to make this documentation more explicit. Currently, there are two distinct part that seem to be logically linked, but actually aren't.

To be more explicit, the two parts are above and below "The Versions app never uses more than 50%". I think we should explain in more details what those paragraphs are referring to. This first one is to limit the amount of versions depending on a given time window, and the second one is to prevent versions from taking more than 50% of the user's free space.

Adding headers would be a good start.

Then please add screenshots.

Comment on lines +71 to +72
Auto-expiry follows the same retention principles but may apply them more flexibly, allowing minor deviations when
necessary to maintain adequate free storage space.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This needs to be more explicit on what kind of deviations. This is deterministic, so we should be able to describe it.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants