`admin media delete-past-remote-media` has confusing time specifiers #1134

New issue

Open

opened 2025-10-22 11:00:56 +00:00 by nex · 1 comment

nex commented

2025-10-22 11:00:56 +00:00

Owner

--before, --after, --newer-than, --older-than are all ambiguous and continue to cause issues with trying to grasp how this command works. Rephrasing the arguments so that they are less ambiguous and easier to understand a glance will go a long way to making sure people don't lose data.

Please suggest the arguments you think would best make sense for this command because it seems nobody can quite get it right.

`--before`, `--after`, `--newer-than`, `--older-than` are all ambiguous and continue to cause issues with trying to grasp how this command works. Rephrasing the arguments so that they are less ambiguous and easier to understand a glance will go a long way to making sure people don't lose data. Please suggest the arguments you think would best make sense for this command because it seems nobody can quite get it right.

nex added the

Help wanted

Matrix/Administration

labels

2025-10-22 11:00:56 +00:00

oddlid commented

2025-10-22 11:07:46 +00:00

Contributor

I think both variants are fine, as long as the doc comments state clearly what each do.
As mentioned in the chat, I read --before and --older-than as deleting all files created before and up to that given point in time. And --after / --newer-than as delete all files created at the given point in time and later, up until right now.

Another way, could be to have a --span argument, or similar, where you specify an inclusive time range to match against. But I guess that could be confusing as well.

I think both variants are fine, as long as the doc comments state clearly what each do. As mentioned in the chat, I read `--before` and `--older-than` as deleting all files created before and up to that given point in time. And `--after` / `--newer-than` as delete all files created at the given point in time and later, up until right now. Another way, could be to have a `--span` argument, or similar, where you specify an inclusive time range to match against. But I guess that could be confusing as well.

oddlid referenced this issue

2025-10-23 07:37:23 +00:00

fix(#1134): Update docs and implementation of admin media delete-past-remote-media #1136

No milestone

No project

No assignees

2 participants

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

continuwuation/continuwuity#1134

No description provided.

Rows
Columns

admin media delete-past-remote-media has confusing time specifiers #1134

`admin media delete-past-remote-media` has confusing time specifiers #1134