akolpakov / django-unused-media

Remove unused media files from Django project
MIT License
124 stars 24 forks source link

Default action should be uppercase #4

Closed saily closed 7 years ago

saily commented 7 years ago

Action triggered by hitting Enter without input should be uppercase, which should show uppercase N instead of Y.

As is:

$ .../manage.py  cleanup_unused_media
These files will be deleted:
screenshots/Screen_Shot_2017-01-16_at_10.56.29.png
Are you sure you want to remove 1 unused files? (Y/n)

Should be:

$ .../manage.py  cleanup_unused_media
These files will be deleted:
screenshots/Screen_Shot_2017-01-16_at_10.56.29.png
Are you sure you want to remove 1 unused files? (y/N)
akolpakov commented 7 years ago

What is the reason behind? Reason to have (Y/n) is that shift + y prevent from occasionally deleting more than just y

For deleting without confirmation, please use --noinput flag

saily commented 7 years ago

default uppercase is what *nix users expect.

Am 16.01.2017 um 17:37 schrieb Andrey notifications@github.com:

What is the reason behind? Reason to have (Y/n) is that shift + y prevent from occasionally deleting more than just y

For deleting without confirmation, please use --noinput flag

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

akolpakov commented 7 years ago

Fixed. Will be in future versions