Do not log an error for "Document not found." This is a normal operation and just becomes noise in the error logs. It might be an error for the calling code, but that's something the caller must decide whether to log.
The query service should just return whatever value it found. It should not be throwing ApiErrorException with a status code. That logic belongs in the controller.
Issue description
The logic for GetById should be changed.
ApiErrorException
with a status code. That logic belongs in the controller.What's the current functionality?
What's the updated acceptance criteria?
Additional details / screenshot
Related Tickets