I will rarely reject a PR for using them but I prefer not using decorators for this. See our internal discussion.
No need to error if a category/tag doesn’t have a chat_channel. I’d rather just return nil and handle it properly on the client.
I understand but I’ve always found it weird from an API perspective. A successful 200 response yet the payload does not contain what I requested for.
I don’t agree; a nil chat channel is expected behavior, and I don’t want to serve a 404 error on expected behavior