-
Notifications
You must be signed in to change notification settings - Fork 5
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
Review reported Tasks #42
Conversation
I just tried to report "coding-challenges". But theres 1. no report button and 2. I get responses of "not authorized" || "permission_denied" when reporting it in SwaggerUI even tough I'm authorized in with bearer. (tried it with admin & normal-user, before solving and after) |
Have you solved the subtask before trying to report it? Currently this is a requirement, but we might have to rethink that later |
Yes I did solve it with 2 accounts and tried to report it but that did not work |
Did you rate the subtask after solving it? Because at the moment you can only rate or report it, but not both (probably also needs to be changed later) |
I have not found any ability to rate or report a coding-challenge with the current UI |
Ok, then it is probably not implemented yet. Could you post the exact http request you are sending to the backend to report the coding challenge (you can just copy paste the curl command from swagger)? And also the response please? |
Yes sure! |
@Defelo Are there new Code-style guidelines because that check failed? |
Yes, we recently added eslint to enforce a consistent code style (#51). You should be able to fix most (if not all) of these errors with |
Preview deployed to https://c42c7158.academy-admin-preview.pages.dev (total size: 3.9M) |
Description
Issue 1: Missing Task Creator Information
Issue 2: Report Details Page - Missing & Incorrect Information
The report details page should display all essential report details. (Reported By, Reason, Comment, Reported At, and Creator of the task)
Locales for added essential report details
The "Options" section should only display the original quiz answer options without any additional text.
Introduce new action buttons as described above. ("Delete Task", "Adjust Task", "Ban Creator", and "Ban Reporter".)
Locales for new Buttons
Issue 3: Report Archiving and Revisiting
Problems with solving Issue 3 in the Frontend
After resolving a reported task, this task is no longer returned from getReportetTaskList(). It would be beneficial if these tasks still appeared in the list but with an additional key indicating their report state. Additionally, it would be useful to incorporate a query parameter in the get call, allowing for the retrieval of either 'open', 'resolved', or 'all' reported tasks.
Be careful if testing the Buttons in the Dev Environment as they are fully functional
Type of change
Fixes Bootstrap-Academy/Bootstrap-Academy#34
My Bootstrap Academy username: