-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Peer rerview deletion requests: Include a link to the policy page #4801
Comments
Hello, can I try this issue? |
@u7253519 Thanks! I updated the links above. The first 3 ones are the same URL, but they may be different in the future. |
@u7253519 Any progress on this? 🙂 |
Hi, I am one of the students who wanted to work on this issue. |
No reply from u7253519. |
Thank you! We intend to start working on it as soon as possible since the assignment is due next week. |
Cool! Please let us know about your progress by posting here at least once a week, thanks a good luck! |
Me and the group have sat down and checked the issue. The file deleteHelper seems to be the one that sends the request and we have found a part that adds the delete reason text. Although we have some questions: The first one: We dont know how to simulate the deletion request, we tried to nominate our own picture for deletion but it did nothing to showcase the problem. The second one: Is adding the link all we really have to do? It feels like we can just add the link where the list is populated. The third one: deleteHelper does not have Freedom of panorama as a deletionRequest as we can see? |
Thanks! |
We made a quick modification that we believe will solve the issue. However, we were not able to simulate the process of showing the alert. Thank you for assigning us this issue but the course assignment connected to this issue is done which means that this will be as far as we contribute. If you want we can pull request what we have done however we believe someone has to simulate the alert more and test them before making the change |
Understood, thank you! :-) |
People to whom deletion requests are sent might not be able to understand the language in which the request in written.
So, we should include a link to the policy page, so that they get at least an idea of what it is about.
Policy pages for each deletion reason:
The text was updated successfully, but these errors were encountered: