{"action":"created","issue":{"url":"https://api.github.com/repos/libgraviton/graviton/issues/205","labels_url":"https://api.github.com/repos/libgraviton/graviton/issues/205/labels{/name}","comments_url":"https://api.github.com/repos/libgraviton/graviton/issues/205/comments","events_url":"https://api.github.com/repos/libgraviton/graviton/issues/205/events","html_url":"https://github.com/libgraviton/graviton/pull/205","id":97629751,"number":205,"title":"add deep fields in showcase test","user":{"login":"narcoticfresh","id":245441,"avatar_url":"https://avatars.githubusercontent.com/u/245441?v=3","gravatar_id":"","url":"https://api.github.com/users/narcoticfresh","html_url":"https://github.com/narcoticfresh","followers_url":"https://api.github.com/users/narcoticfresh/followers","following_url":"https://api.github.com/users/narcoticfresh/following{/other_user}","gists_url":"https://api.github.com/users/narcoticfresh/gists{/gist_id}","starred_url":"https://api.github.com/users/narcoticfresh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/narcoticfresh/subscriptions","organizations_url":"https://api.github.com/users/narcoticfresh/orgs","repos_url":"https://api.github.com/users/narcoticfresh/repos","events_url":"https://api.github.com/users/narcoticfresh/events{/privacy}","received_events_url":"https://api.github.com/users/narcoticfresh/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-07-28T06:50:07Z","updated_at":"2015-07-28T08:46:37Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/libgraviton/graviton/pulls/205","html_url":"https://github.com/libgraviton/graviton/pull/205","diff_url":"https://github.com/libgraviton/graviton/pull/205.diff","patch_url":"https://github.com/libgraviton/graviton/pull/205.patch"},"body":"this adds deeply nested fields to the ShowCase test\r\n\r\n@hairmare This one actually highlights a week spot of the Graviton behavior at the moment. As you can see, I needed to add empty fields to `showcase-minimal.json`\r\n\r\nIt seems that Graviton\r\n* Always renders out empty arrays, even if they are subproperties (n levels) of actually empty properties that nobody set in the first place (see `deepNestedArray`) \r\n* Somehow renders out the deep field `deepNestedProperty.thisis.one.goes.very.deep.down.thats` as array (it should be an object). Also it shouldn't render it at all cause nobody sets it.\r\n\r\nBoth fields mentioned in the bullet points above get rendered by Graviton in after the POST creation *regardless if they have been passed in by the client* - they just get rendered no matter what. \r\n\r\nThis goes contrary to the concept of \"not rendering null values\" somehow as we're rendering out stuff that isn't set. Should we address this or leave it as-is?"},"comment":{"url":"https://api.github.com/repos/libgraviton/graviton/issues/comments/125507112","html_url":"https://github.com/libgraviton/graviton/pull/205#issuecomment-125507112","issue_url":"https://api.github.com/repos/libgraviton/graviton/issues/205","id":125507112,"user":{"login":"hairmare","id":116588,"avatar_url":"https://avatars.githubusercontent.com/u/116588?v=3","gravatar_id":"","url":"https://api.github.com/users/hairmare","html_url":"https://github.com/hairmare","followers_url":"https://api.github.com/users/hairmare/followers","following_url":"https://api.github.com/users/hairmare/following{/other_user}","gists_url":"https://api.github.com/users/hairmare/gists{/gist_id}","starred_url":"https://api.github.com/users/hairmare/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hairmare/subscriptions","organizations_url":"https://api.github.com/users/hairmare/orgs","repos_url":"https://api.github.com/users/hairmare/repos","events_url":"https://api.github.com/users/hairmare/events{/privacy}","received_events_url":"https://api.github.com/users/hairmare/received_events","type":"User","site_admin":false},"created_at":"2015-07-28T08:46:37Z","updated_at":"2015-07-28T08:46:37Z","body":"@narcoticfresh Probably. I can't check right now without backing out of a largish change..."}}