feat: adds Tyran's "GET", "POST", and "DELETE" request for Media Items #733
Conversation
RiceViz
changed the title
anthonydmays
left a comment
anthonydmays
left a comment
There was a problem hiding this comment.
🎓 Automated Grading Report
Student: Tyran Rice
Date: 10/7/2025
Feedback
The submission demonstrates a good understanding of APIs and the HTTP protocol by implementing the required endpoints in the MediaItemsController.java file. The student has correctly added the necessary imports and annotations to handle HTTP requests. The getItemsById, addItem, and deleteItem methods have been implemented to interact with the library and respond with appropriate HTTP responses. However, there are some discrepancies compared to the sample solution. For instance, the getItemsById method returns a GetMediaItemsResponse instead of a MediaItemResponse, which is inconsistent with the sample solution. Additionally, the addItem method returns a Map<String, MediaItemResponse> instead of a CreateMediaItemResponse, which deviates from the expected response structure. These differences may affect the functionality and compatibility with existing unit tests. Technically, the code is mostly well-formatted and syntactically correct, but there are minor inconsistencies in the response types and exception handling. Overall, the submission meets most of the functional and technical requirements but could benefit from aligning more closely with the sample solution.
This is an automated preliminary review. Please review and adjust before finalizing.
| } | ||
|
|
||
| @GetMapping("/items/{id}") | ||
| public ResponseEntity<GetMediaItemsResponse> getItemsById(@PathVariable String id) { |
There was a problem hiding this comment.
Should be getItemById and should just return a MediaItemResponse.
| Map<String, MediaItemResponse> response = Map.of("item", responseItem); | ||
|
|
||
| return ResponseEntity.ok(response); | ||
| } catch (IllegalArgumentException e) { |
2 participants
This PR adds Web API's to make HTTP calls "GET", "POST" and "DELETE" for Media Items.
Lessons Learned:
Challenges: