{"action":"created","issue":{"url":"https://api.github.com/repos/Ethelo/kronky/issues/5","repository_url":"https://api.github.com/repos/Ethelo/kronky","labels_url":"https://api.github.com/repos/Ethelo/kronky/issues/5/labels{/name}","comments_url":"https://api.github.com/repos/Ethelo/kronky/issues/5/comments","events_url":"https://api.github.com/repos/Ethelo/kronky/issues/5/events","html_url":"https://github.com/Ethelo/kronky/pull/5","id":291307582,"node_id":"MDExOlB1bGxSZXF1ZXN0MTY0OTIzMzA3","number":5,"title":"Add support for nested changeset error messages","user":{"login":"simonprev","id":464900,"node_id":"MDQ6VXNlcjQ2NDkwMA==","avatar_url":"https://avatars2.githubusercontent.com/u/464900?v=4","gravatar_id":"","url":"https://api.github.com/users/simonprev","html_url":"https://github.com/simonprev","followers_url":"https://api.github.com/users/simonprev/followers","following_url":"https://api.github.com/users/simonprev/following{/other_user}","gists_url":"https://api.github.com/users/simonprev/gists{/gist_id}","starred_url":"https://api.github.com/users/simonprev/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/simonprev/subscriptions","organizations_url":"https://api.github.com/users/simonprev/orgs","repos_url":"https://api.github.com/users/simonprev/repos","events_url":"https://api.github.com/users/simonprev/events{/privacy}","received_events_url":"https://api.github.com/users/simonprev/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":13,"created_at":"2018-01-24T18:10:48Z","updated_at":"2018-10-09T12:44:58Z","closed_at":null,"author_association":"NONE","pull_request":{"url":"https://api.github.com/repos/Ethelo/kronky/pulls/5","html_url":"https://github.com/Ethelo/kronky/pull/5","diff_url":"https://github.com/Ethelo/kronky/pull/5.diff","patch_url":"https://github.com/Ethelo/kronky/pull/5.patch"},"body":"This adds basic support for errors in nested changesets. I’m open to change the formatting for nested fields, currently it’s: `association.field` but it could also be `associationField` or something else. The dot notation fitted our use case.\r\n\r\n## Results in a GraphQL payload\r\n\r\n_Client belongs to tutor with validation for the tutor association_\r\n\r\n```elixir\r\n%{\r\n  data: %{\r\n    \"updateClient\" => %{\r\n      \"messages\" => [\r\n        %{\"code\" => \"required\", \"field\" => \"firstname\"},\r\n        %{\"code\" => \"required\", \"field\" => \"tutor.email\"},\r\n        %{\"code\" => \"required\", \"field\" => \"tutor.firstname\"},\r\n        %{\"code\" => \"required\", \"field\" => \"tutor.lastname\"},\r\n      ]\r\n    }\r\n  }\r\n}\r\n```\r\n\r\nI also fixed a warning and a failing test in `test/test_helper_test.exs`"},"comment":{"url":"https://api.github.com/repos/Ethelo/kronky/issues/comments/428177291","html_url":"https://github.com/Ethelo/kronky/pull/5#issuecomment-428177291","issue_url":"https://api.github.com/repos/Ethelo/kronky/issues/5","id":428177291,"node_id":"MDEyOklzc3VlQ29tbWVudDQyODE3NzI5MQ==","user":{"login":"quolpr","id":7958527,"node_id":"MDQ6VXNlcjc5NTg1Mjc=","avatar_url":"https://avatars0.githubusercontent.com/u/7958527?v=4","gravatar_id":"","url":"https://api.github.com/users/quolpr","html_url":"https://github.com/quolpr","followers_url":"https://api.github.com/users/quolpr/followers","following_url":"https://api.github.com/users/quolpr/following{/other_user}","gists_url":"https://api.github.com/users/quolpr/gists{/gist_id}","starred_url":"https://api.github.com/users/quolpr/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/quolpr/subscriptions","organizations_url":"https://api.github.com/users/quolpr/orgs","repos_url":"https://api.github.com/users/quolpr/repos","events_url":"https://api.github.com/users/quolpr/events{/privacy}","received_events_url":"https://api.github.com/users/quolpr/received_events","type":"User","site_admin":false},"created_at":"2018-10-09T12:44:58Z","updated_at":"2018-10-09T12:44:58Z","author_association":"NONE","body":"@ssbb could you please merge it 🙏 "}}
{"actor":{"display_login":"quolpr"}}