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.

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

Allow screenshot uploaders to delete their own screenshots #99

Open
glennlunder opened this issue Mar 24, 2014 · 2 comments
Open

Allow screenshot uploaders to delete their own screenshots #99

glennlunder opened this issue Mar 24, 2014 · 2 comments

Comments

@glennlunder
Copy link

@glennlunder Going over the logs for the last night, and I noticed something - http://demozoo.org/productions/534/ - now, I know for a fact that I added those screenshots, likely before we had logging. In this PARTICULAR case, the user tags = Eladamri, who coded the prod, so I expect he had good reason, but the fact that he could just up and delete screenshots like that worries me. Do we have any sort of way to recover screenshots that are deleted, and should we try to tighten that bit up in such a way that for instance only the person who uploaded screenshots and admins can delete?

@gasman Oh. I could have sworn that was always supposed to be admin only - oh well, it is now... Allowing the uploader to delete their own screenshots is a good idea, but would be non-trivial to implement (in particular, it's going to be tricky to come up with a non-confusing UI that makes it clear why you can delete some screenshots and not others) so I'll ticket that one up for later.

@gasman gasman added enhancement and removed bug labels Jul 19, 2014
@glennlunder glennlunder added this to the real-soon-now milestone May 23, 2016
@nswaldman nswaldman added this to Backlog in Screenshot Management Apr 22, 2018
@gasman gasman added this to To do in Screenshot management Apr 22, 2018
@asbjornu
Copy link
Collaborator

Yep, I just discovered this. Perhaps allowing users to choose the "key" screenshot for a production would alleviate some of the need for screenshot deletion?

@glennlunder
Copy link
Author

There's a separate ticket for that - #12

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

No branches or pull requests

3 participants