This repository has been archived by the owner on Feb 8, 2018. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 308
expose UI for deactivating account #664
Labels
Comments
chadwhitacre
added a commit
that referenced
this issue
Feb 20, 2013
chadwhitacre
added a commit
that referenced
this issue
Feb 25, 2013
Deleting an account is of course a big deal and shouldn't be too easy to do. Leading with a button makes it feel too available as an option. Linking to a page with more info is better.
chadwhitacre
added a commit
that referenced
this issue
Mar 8, 2013
chadwhitacre
added a commit
that referenced
this issue
Mar 8, 2013
chadwhitacre
added a commit
that referenced
this issue
Mar 8, 2013
chadwhitacre
added a commit
that referenced
this issue
Mar 8, 2013
chadwhitacre
added a commit
that referenced
this issue
Mar 8, 2013
This actually uncovered a bug in Aspen where empty POST bodies cause an exception in the request parser. We needed the csrf_token anyway, of course, but really we're masking an underlying bug (to work around it we would have needed *some* field, anyway).
chadwhitacre
added a commit
that referenced
this issue
Mar 8, 2013
Is this right? This can't be right. Is this right?
#54, whence this came, is done. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
See #586 for wiring.
The text was updated successfully, but these errors were encountered: