Introducing the mbin:images:delete command #1394
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
melroy89
added
enhancement
BentiGorlich
requested changes
Jan 28, 2025
Member
BentiGorlich
left a comment
BentiGorlich
left a comment
There was a problem hiding this comment.
Some remarks. Additionally I would like to see a counter at the end how many images were deleted, but that is just icing on the cake :)
melroy89
force-pushed
the
delete_old_images
branch
from
66390d3 to
06b0a90
Compare
Co-authored-by: BentiGorlich <[email protected]>
melroy89
force-pushed
the
delete_old_images
branch
from
06b0a90 to
6439262
Compare
BentiGorlich
requested changes
Jan 29, 2025
Member
BentiGorlich
left a comment
BentiGorlich
left a comment
There was a problem hiding this comment.
Just small annotations. Otherwise it looks fine :)
Co-authored-by: BentiGorlich <[email protected]>
BentiGorlich
approved these changes
Jan 29, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
monthsAgoarg is set to 12 months (1 year)typearg is set to "all", meaning delete all images from threads, threads comments, posts and post comments, with the exceptions of users. Since I believe users can still be activate after 12 months, right ;)..In my case, each time running
./bin/console mbin:images:delete, will delete around 1GB of data from my media directory. Just for info.