Skip to content

Conversation

@tmcrockett252
Copy link
Contributor

  • Add GET /items endpoint to retrieve all media items
  • Add GET /items/{id} endpoint to retrieve media item by ID
  • Add POST /items endpoint for creating media items (incomplete)
  • Add DELETE /items/{id} endpoint stub (incomplete)
  • Implement search functionality using SearchCriteria
  • Add proper 404 handling for single item retrieval

It's still a work in progress.

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: Talia Crockett
Date: 10/7/2025

Feedback

The submission demonstrates a solid understanding of the basic requirements for implementing API endpoints in Java using Spring Boot. The student has successfully implemented the necessary HTTP methods: GET, POST, and DELETE for managing media items. The code modifications include handling requests and responses appropriately, and the use of annotations such as @GetMapping, @PostMapping, and @DeleteMapping is correct. However, there are some areas for improvement. In the GET method for retrieving a single item, the response type should be MediaItemResponse instead of GetMediaItemsResponse to match the sample solution. Additionally, the error handling in the DELETE method could be improved by directly checking if the item exists using library.hasMediaItem() instead of searching for it first. The code is generally well-formatted and syntactically correct, but there are minor inconsistencies in the use of variable naming and response handling that could be refined. Overall, the submission meets most of the functional and technical requirements, but there are opportunities for refinement to fully align with the sample solution.


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

@anthonydmays anthonydmays reopened this Oct 8, 2025
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