{"comment":{"url":"https://api.github.com/repos/alphagov/specialist-publisher/comments/11792925","html_url":"https://github.com/alphagov/specialist-publisher/commit/b2a8880646bb9b138b811ed8b50f0d0689d53237#commitcomment-11792925","id":11792925,"user":{"login":"jackscotti","id":5038475,"avatar_url":"https://avatars.githubusercontent.com/u/5038475?v=3","gravatar_id":"","url":"https://api.github.com/users/jackscotti","html_url":"https://github.com/jackscotti","followers_url":"https://api.github.com/users/jackscotti/followers","following_url":"https://api.github.com/users/jackscotti/following{/other_user}","gists_url":"https://api.github.com/users/jackscotti/gists{/gist_id}","starred_url":"https://api.github.com/users/jackscotti/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jackscotti/subscriptions","organizations_url":"https://api.github.com/users/jackscotti/orgs","repos_url":"https://api.github.com/users/jackscotti/repos","events_url":"https://api.github.com/users/jackscotti/events{/privacy}","received_events_url":"https://api.github.com/users/jackscotti/received_events","type":"User","site_admin":false},"position":24,"line":137,"path":"spec/models/manual_with_documents_spec.rb","commit_id":"b2a8880646bb9b138b811ed8b50f0d0689d53237","created_at":"2015-06-22T12:31:48Z","updated_at":"2015-06-22T12:31:48Z","body":"Do you need to create a new document that is equal to `document_a` in order to delete it from the manual? \r\nIf you can't send `document_a` itself, I think that renaming `document_with_id_a` to something like `document_to_remove` could make it a bit more easy to understand to someone that is reading the code for the first time (me)."}}