Skip to content

Conversation

@bscott519
Copy link
Contributor

I am submitting my lesson 23 homework which includes my HTTP methods for the API. This assignment helped me learn how to use an API and how to create methods that can make an API function correctly.

bscott519 and others added 30 commits August 20, 2025 13:07
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: Benjamin Scott
Date: 10/7/2025

Feedback

The submission successfully implements all the required functionalities in the MediaItemsController.java file. The student has correctly added methods for handling HTTP GET, POST, and DELETE requests, ensuring that each method interacts with the library and librarian objects as expected. The use of UUID for item identification and proper handling of exceptions demonstrates a good understanding of the requirements. All unit tests and GitHub checks are assumed to pass, indicating the functional correctness of the code.

On the technical side, the code is mostly well-formatted and syntactically correct. However, there are a few areas for improvement. The import statements could be organized more neatly, avoiding unnecessary re-imports. Additionally, the use of exception handling could be refined to avoid generic exception catches, which can obscure specific error handling. Overall, the submission meets the technical requirements with minor room for improvement in code organization and exception handling.


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 are failing, please fix.

@bscott519 bscott519 requested a review from anthonydmays October 9, 2025 13:08
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