{"action":"created","issue":{"url":"https://api.github.com/repos/looplab/eventhorizon/issues/213","repository_url":"https://api.github.com/repos/looplab/eventhorizon","labels_url":"https://api.github.com/repos/looplab/eventhorizon/issues/213/labels{/name}","comments_url":"https://api.github.com/repos/looplab/eventhorizon/issues/213/comments","events_url":"https://api.github.com/repos/looplab/eventhorizon/issues/213/events","html_url":"https://github.com/looplab/eventhorizon/pull/213","id":338381779,"node_id":"MDExOlB1bGxSZXF1ZXN0MTk5MzI4MTQx","number":213,"title":"Fix dynamo event data structure","user":{"login":"mcarriere","id":25467653,"node_id":"MDQ6VXNlcjI1NDY3NjUz","avatar_url":"https://avatars1.githubusercontent.com/u/25467653?v=4","gravatar_id":"","url":"https://api.github.com/users/mcarriere","html_url":"https://github.com/mcarriere","followers_url":"https://api.github.com/users/mcarriere/followers","following_url":"https://api.github.com/users/mcarriere/following{/other_user}","gists_url":"https://api.github.com/users/mcarriere/gists{/gist_id}","starred_url":"https://api.github.com/users/mcarriere/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mcarriere/subscriptions","organizations_url":"https://api.github.com/users/mcarriere/orgs","repos_url":"https://api.github.com/users/mcarriere/repos","events_url":"https://api.github.com/users/mcarriere/events{/privacy}","received_events_url":"https://api.github.com/users/mcarriere/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-07-04T22:01:54Z","updated_at":"2018-09-06T06:42:54Z","closed_at":null,"author_association":"NONE","pull_request":{"url":"https://api.github.com/repos/looplab/eventhorizon/pulls/213","html_url":"https://github.com/looplab/eventhorizon/pull/213","diff_url":"https://github.com/looplab/eventhorizon/pull/213.diff","patch_url":"https://github.com/looplab/eventhorizon/pull/213.patch"},"body":"# Summary\r\nThe previous version would convert the raw event data to a map of map containing all but one null values. Seems like the issue was with how the MarshalMap function handles the data structure. I could not find a clean way to fix this without using the github.com/guregu/dynamo library that already handles this kind of marshalling.\r\n\r\nFrom now on the RawData field contains a map of only the real value (i.e a map of strings and numbers instead of a map of map of mostly null values)\r\n\r\n# How to test\r\nSaving any event should demonstrate the change since the RawData field won't contain maps with a bunch of null values anymore\r\n\r\n# Issue\r\n\r\n\r\n# Notes\r\n"},"comment":{"url":"https://api.github.com/repos/looplab/eventhorizon/issues/comments/418982797","html_url":"https://github.com/looplab/eventhorizon/pull/213#issuecomment-418982797","issue_url":"https://api.github.com/repos/looplab/eventhorizon/issues/213","id":418982797,"node_id":"MDEyOklzc3VlQ29tbWVudDQxODk4Mjc5Nw==","user":{"login":"maxekman","id":821518,"node_id":"MDQ6VXNlcjgyMTUxOA==","avatar_url":"https://avatars0.githubusercontent.com/u/821518?v=4","gravatar_id":"","url":"https://api.github.com/users/maxekman","html_url":"https://github.com/maxekman","followers_url":"https://api.github.com/users/maxekman/followers","following_url":"https://api.github.com/users/maxekman/following{/other_user}","gists_url":"https://api.github.com/users/maxekman/gists{/gist_id}","starred_url":"https://api.github.com/users/maxekman/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/maxekman/subscriptions","organizations_url":"https://api.github.com/users/maxekman/orgs","repos_url":"https://api.github.com/users/maxekman/repos","events_url":"https://api.github.com/users/maxekman/events{/privacy}","received_events_url":"https://api.github.com/users/maxekman/received_events","type":"User","site_admin":false},"created_at":"2018-09-06T06:42:54Z","updated_at":"2018-09-06T06:42:54Z","author_association":"MEMBER","body":"Hey @mcarriere! Sorry for the delay, have been super busy after vacation. I'm willing to merge this if you can rebase and add the dep using Go 1.11 instead."}}
{"actor":{"display_login":"maxekman"}}