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

Do not delete comments, apps #113

Open
derrell opened this issue Sep 24, 2011 · 5 comments
Open

Do not delete comments, apps #113

derrell opened this issue Sep 24, 2011 · 5 comments
Assignees
@piersquared
Labels
SWE

Comments

@derrell
Copy link
Owner

derrell commented Sep 24, 2011

Instead of deleting comments and apps from the database, add a new status value "Deleted" and mark the comment or app with this status.

This is not as trivial as it sounds, because the code must be inspected throughout to ensure that queries for comments and apps ignore ones that are marked as "Deleted".
@ghost ghost assigned piersquared Nov 5, 2011
@derrell
Copy link
Owner Author

derrell commented Nov 5, 2011

Paul, to you.

@piersquared
Copy link
Collaborator

thanks
so status value "Deleted" would prevent user from seeing it, but not administrators from seeing it if they wanted to.

@piersquared
Copy link
Collaborator

status: just started should take 5-10 hours to finish this issues.

@piersquared
Copy link
Collaborator

Status update I looked at it and have no internet till Thursdays thanks to Verizon

@derrell
Copy link
Owner Author

derrell commented Dec 5, 2011

Consider changing "Delete" to "Unpublish" to make it more clear that it's not actually being deleted.

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

@piersquared piersquared

Labels
SWE
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@derrell @piersquared