Skip to content

Conversation

@DaFDE31
Copy link
Contributor

@DaFDE31 DaFDE31 commented Oct 6, 2025

This pr is to add methods to the MediaItemsController that will take HTTP requests and perform methods accordingly.

  • GET /items gets a list of all media items.
  • Post /items posts a new media item unless the given item is not of an existing type.
  • GET /items/:id gets the media item of the corresponding id
  • DELETE /items/:id deletes the media item of the corresponding id

Copy link
Contributor

@anthonydmays anthonydmays left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎓 Automated Grading Report

Student: Danielson Adjocy
Date: 10/7/2025

Feedback

The submission demonstrates a good understanding of the requirements to modify the MediaItemsController.java file. The student successfully implemented the necessary endpoints for creating, retrieving, and deleting media items, which aligns with the functional requirements. However, there are some discrepancies compared to the sample solution. For instance, the getItem method returns a GetMediaItemsResponse instead of a MediaItemResponse, which deviates from the expected behavior. Additionally, the deleteItem method uses a search to find the item before deletion, which is a valid approach but differs from the sample solution's use of UUID parsing and direct library methods. These differences suggest a partial completion of the requirements, hence the score of 4 for functionality.

On the technical side, the code is mostly well-formatted and syntactically correct. The use of Map for error responses is a good practice for providing detailed feedback. However, the import statements are slightly disorganized, with some unnecessary additions and reordering. The code changes also include redundant exception handling in the deleteItem method, which could be streamlined. These minor issues lead to a technical score of 4.

Overall, the submission is strong but could benefit from closer alignment with the sample solution and some technical refinements.


This is an automated preliminary review. Please review and adjust before finalizing.

@anthonydmays anthonydmays reopened this Oct 8, 2025
Copy link
Contributor

@anthonydmays anthonydmays left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Checks not passing, please fix.

@DaFDE31 DaFDE31 requested a review from anthonydmays October 8, 2025 20:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants