Add option to delete photo (fix bug 689365) #64

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
Member

tofumatt commented Nov 28, 2011

Oh hey: you can delete your photo now. Is there a cool mock file test thing for Django we can test this with, or is this better left to automation? (@m8ttyB -- what you say?)

Member

davedash commented Nov 28, 2011

tests, otherwise it looks great.

Member

tofumatt commented Nov 28, 2011

Also this fails if the user doesn't have a profile photo but sends the key anyway. I'll hide the option if their photo is blank and make sure we don't try to delete it if it doesn't exist.

Member

tofumatt commented Nov 28, 2011

Merged in 3eb4453.

@tofumatt tofumatt closed this Nov 28, 2011

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