Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Users can't be deleted #1163

Closed
vitorbaptista opened this issue Aug 7, 2013 · 2 comments
Closed

Users can't be deleted #1163

vitorbaptista opened this issue Aug 7, 2013 · 2 comments
Assignees
@vitorbaptista

Comments

@vitorbaptista
Copy link
Contributor

After a user was created, there's no way to delete it, other than going to the database and removing its entry.
vitorbaptista added a commit that referenced this issue Aug 7, 2013
@vitorbaptista
[#1163] Users can be deleted
e430f95 
To do this, I've configured the User model to be stateful using vdm.sqlalchemy.
Right now, there're two states: active and deleted. If a user is deleted, he
can't login, and is unauthorized to do anything. She also doesn't appear in the
user's list anymore, but you can still access her profile page, if you know her
username.

If she was logged in when her user was deleted, the next time she goes into
CKAN, she'll be logged off. Unfortunately, there's not a useful message like
"Your user has been deleted." Yet.

There's no way to undelete a user, but it should be simply creating an
action to set her state to active.
@ghost ghost assigned vitorbaptista Aug 7, 2013
vitorbaptista added a commit that referenced this issue Aug 7, 2013
@vitorbaptista
[#1163] Users can be deleted
d448c8b 
To do this, I've configured the User model to be stateful using vdm.sqlalchemy.
Right now, there're two states: active and deleted. If a user is deleted, he
can't login, and is unauthorized to do anything. She also doesn't appear in the
user's list anymore, but you can still access her profile page, if you know her
username.

If she was logged in when her user was deleted, the next time she goes into
CKAN, she'll be logged off. Unfortunately, there's not a useful message like
"Your user has been deleted." Yet.

There's no way to undelete a user, but it should be simply creating an
action to set her state to active.
vitorbaptista added a commit that referenced this issue Aug 7, 2013
@vitorbaptista
[#1163] Users can be deleted
4c0557b 
To do this, I've configured the User model to be stateful using vdm.sqlalchemy.
Right now, there're two states: active and deleted. If a user is deleted, he
can't login, and is unauthorized to do anything. She also doesn't appear in the
user's list anymore, but you can still access her profile page, if you know her
username.

If she was logged in when her user was deleted, the next time she goes into
CKAN, she'll be logged off. Unfortunately, there's not a useful message like
"Your user has been deleted." Yet.

There's no way to undelete a user, but it should be simply creating an
action to set her state to active.
vitorbaptista added a commit that referenced this issue Aug 7, 2013
@vitorbaptista
[#1163] Users can be deleted
ae9b9d8 
To do this, I've configured the User model to be stateful using vdm.sqlalchemy.
Right now, there're two states: active and deleted. If a user is deleted, he
can't login, and is unauthorized to do anything. She also doesn't appear in the
user's list anymore, but you can still access her profile page, if you know her
username.

If she was logged in when her user was deleted, the next time she goes into
CKAN, she'll be logged off. Unfortunately, there's not a useful message like
"Your user has been deleted." Yet.

There's no way to undelete a user, but it should be simply creating an
action to set her state to active.
vitorbaptista added a commit that referenced this issue Aug 9, 2013
@vitorbaptista
[#1163] Use ckan.model.State instead of vdm.sqlalchemy.State
35112b2 
This makes us a bit less tied to vdm.
vitorbaptista added a commit that referenced this issue Aug 9, 2013
@vitorbaptista
[#1163] Use ckan.model.core.State instead of ckan.model.State
f2615de 
With this change, we're able to avoid having to load ckan.model inside methods,
to avoid circular dependencies.
vitorbaptista added a commit that referenced this issue Aug 18, 2013
@vitorbaptista
[#1163] Users can be deleted
7c60ba4 
To do this, I've configured the User model to be stateful using vdm.sqlalchemy.
Right now, there're two states: active and deleted. If a user is deleted, he
can't login, and is unauthorized to do anything. She also doesn't appear in the
user's list anymore, but you can still access her profile page, if you know her
username.

If she was logged in when her user was deleted, the next time she goes into
CKAN, she'll be logged off. Unfortunately, there's not a useful message like
"Your user has been deleted." Yet.

There's no way to undelete a user, but it should be simply creating an
action to set her state to active.
vitorbaptista added a commit that referenced this issue Aug 18, 2013
@vitorbaptista
[#1163] Use ckan.model.State instead of vdm.sqlalchemy.State
ad2de00 
This makes us a bit less tied to vdm.
vitorbaptista added a commit that referenced this issue Aug 18, 2013
@vitorbaptista
[#1163] Use ckan.model.core.State instead of ckan.model.State
d6fde48 
With this change, we're able to avoid having to load ckan.model inside methods,
to avoid circular dependencies.
@kindly kindly closed this as completed Sep 2, 2013
@kindly
Copy link
Contributor

kindly commented Sep 2, 2013

Duplicate of #1163

@vitorbaptista
Copy link
Contributor Author

@kindly I suppose you meant another issue? :P

@nigelbabu nigelbabu mentioned this issue Oct 21, 2013
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vitorbaptista vitorbaptista

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@kindly @vitorbaptista