search

incuna / django-user-management

User management model mixins and api views.
BSD 2-Clause "Simplified" License
57 stars 24 forks source link

Use VerifyEmailViewMixin for the UI version of the view #150

Closed adam-thomas closed 8 years ago

adam-thomas commented 8 years ago

@incuna/backend review please?

adam-thomas commented 8 years ago

bump?

adam-thomas commented 8 years ago

guise plz

meshy commented 8 years ago

https://31.media.tumblr.com/1b67d614deca258a8ae4f2ebeb4dc913/tumblr_inline_n96eetMK2z1r7yvby.gif

adam-thomas commented 8 years ago

Any more comments for this particular PR? (I'll look into returning responses rather than throwing exceptions next.)

adam-thomas commented 8 years ago

@incuna/backend further bumpage (sorry @meshy) :P

kevinetienne commented 8 years ago

I was waiting for a response to https://github.com/incuna/django-user-management/pull/150#discussion_r54437915 but looks good to me

adam-thomas commented 8 years ago

Yeah that's why I was bumping the thread :P Cheers, I can come back to it if there are any further issues.