Skip to content

Conversation

@KerryFerguson04
Copy link
Contributor

This pull request contains my changes for the lesson_23 assignment. Tyran was a big help and helped me understand what was going on in the code (and talked me through on how to approach the assignment). I understand what is happening conceptually, but the syntax escapes me more often than I would like it to. I need to put more time into learning this stuff.

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: Kerry Ferguson
Date: 10/7/2025

Feedback

The submission demonstrates a solid attempt to meet the functional requirements by implementing the necessary endpoints in the MediaItemsController.java file. The student added methods for retrieving, adding, and deleting media items, which aligns with the task's objectives. 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 expected output for retrieving a single item. Additionally, the addItem method returns a Map<String, MediaItemResponse> instead of a CreateMediaItemResponse, which deviates from the expected response structure. These inconsistencies suggest a partial understanding of the requirements, leading to a functional score of 4.

From a technical perspective, the code is mostly syntactically correct and well-formatted, but there are issues with the response types and exception handling. The deleteItem method does not handle exceptions as robustly as the sample solution, which could lead to unhandled scenarios. The imports are slightly disorganized, with some unnecessary additions and rearrangements. These issues indicate a partial attempt to meet the technical requirements, resulting in a technical score of 3.

No stretch criteria were provided or addressed in the submission, so the stretch score is 0.


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