add django-cleanup to remove old images

codeeu / coding-events

A Django app for adding EU Code Week events and showing them on a map.

http://events.codeweek.eu

MIT License

17 stars 36 forks source link

add django-cleanup to remove old images #312

Closed Konviser closed 9 years ago

coveralls commented 9 years ago

Coverage remained the same when pulling 480e100ad20d7452d28adc519a252cddefd4b49f on Konviser:master into 8ccbcb8227317f33584b3f351d86cce36337faa9 on codeeu:master.

goranche commented 9 years ago

please fix the requirements in server-requirements.txt as well

Konviser commented 9 years ago

done

coveralls commented 9 years ago

Coverage remained the same when pulling af59bc127f8b81c9b585c1cd426e78bfdf4eebdb on Konviser:master into 8ccbcb8227317f33584b3f351d86cce36337faa9 on codeeu:master.

goranche commented 9 years ago

:angel: could you please place it in the same line as it's in requirements.txt? that way it's much easier to compare the two files and see the differences

(try opening both files in separate tabs and switch between them)

Konviser commented 9 years ago

yes, I can :)

coveralls commented 9 years ago

Coverage remained the same when pulling ec1290cf3b8db3c20790002c322a8163cdc48c0d on Konviser:master into 8ccbcb8227317f33584b3f351d86cce36337faa9 on codeeu:master.

goranche commented 9 years ago

thank you :)

ercchy commented 9 years ago

@Konviser have you tested this? Does it delete the files?

Konviser commented 9 years ago

yes, it does

ercchy commented 9 years ago

Could we have some tests? Update the ones that already exists.

Konviser commented 9 years ago

we can, i will write/update some

ercchy commented 9 years ago

awesome! You can do it tomorrow and teach the kittens :)

Konviser commented 9 years ago

yep, great idea :)

goranche commented 9 years ago

this means the time should be right to write a script that deletes stale files as well, right?