incuna / django-user-management

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

Add VerifyUserEmailView #146

Closed adam-thomas closed 8 years ago

adam-thomas commented 8 years ago

We use this sort of view all over the place, probably about time we put it in a library. (Turns out #69 agrees with me.) On @maxpeterson's suggestion I put it under a folder called ui, to contrast with api.

@incuna/backend review please? :)

adam-thomas commented 8 years ago

Updated :)

adam-thomas commented 8 years ago

Any more comments?

kevinetienne commented 8 years ago

Looks good but seems travis is taking its time

adam-thomas commented 8 years ago

Standard. I'll keep an eye on it.

LilyFoote commented 8 years ago

You also need to edit tox.ini to add django19.

adam-thomas commented 8 years ago

done!

adam-thomas commented 8 years ago

Can I merge?

adam-thomas commented 8 years ago

Ta :)