{"action":"created","issue":{"url":"https://api.github.com/repos/volCommunity/vol-django/issues/86","repository_url":"https://api.github.com/repos/volCommunity/vol-django","labels_url":"https://api.github.com/repos/volCommunity/vol-django/issues/86/labels{/name}","comments_url":"https://api.github.com/repos/volCommunity/vol-django/issues/86/comments","events_url":"https://api.github.com/repos/volCommunity/vol-django/issues/86/events","html_url":"https://github.com/volCommunity/vol-django/pull/86","id":270836312,"number":86,"title":"Migrate from exposing IDs to UUIDs, support creation of nested jobs (WIP)","user":{"login":"ojongerius","id":1726055,"avatar_url":"https://avatars0.githubusercontent.com/u/1726055?v=4","gravatar_id":"","url":"https://api.github.com/users/ojongerius","html_url":"https://github.com/ojongerius","followers_url":"https://api.github.com/users/ojongerius/followers","following_url":"https://api.github.com/users/ojongerius/following{/other_user}","gists_url":"https://api.github.com/users/ojongerius/gists{/gist_id}","starred_url":"https://api.github.com/users/ojongerius/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ojongerius/subscriptions","organizations_url":"https://api.github.com/users/ojongerius/orgs","repos_url":"https://api.github.com/users/ojongerius/repos","events_url":"https://api.github.com/users/ojongerius/events{/privacy}","received_events_url":"https://api.github.com/users/ojongerius/received_events","type":"User","site_admin":false},"labels":[{"id":703886428,"url":"https://api.github.com/repos/volCommunity/vol-django/labels/enhancement","name":"enhancement","color":"84b6eb","default":true}],"state":"open","locked":false,"assignee":{"login":"ojongerius","id":1726055,"avatar_url":"https://avatars0.githubusercontent.com/u/1726055?v=4","gravatar_id":"","url":"https://api.github.com/users/ojongerius","html_url":"https://github.com/ojongerius","followers_url":"https://api.github.com/users/ojongerius/followers","following_url":"https://api.github.com/users/ojongerius/following{/other_user}","gists_url":"https://api.github.com/users/ojongerius/gists{/gist_id}","starred_url":"https://api.github.com/users/ojongerius/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ojongerius/subscriptions","organizations_url":"https://api.github.com/users/ojongerius/orgs","repos_url":"https://api.github.com/users/ojongerius/repos","events_url":"https://api.github.com/users/ojongerius/events{/privacy}","received_events_url":"https://api.github.com/users/ojongerius/received_events","type":"User","site_admin":false},"assignees":[{"login":"ojongerius","id":1726055,"avatar_url":"https://avatars0.githubusercontent.com/u/1726055?v=4","gravatar_id":"","url":"https://api.github.com/users/ojongerius","html_url":"https://github.com/ojongerius","followers_url":"https://api.github.com/users/ojongerius/followers","following_url":"https://api.github.com/users/ojongerius/following{/other_user}","gists_url":"https://api.github.com/users/ojongerius/gists{/gist_id}","starred_url":"https://api.github.com/users/ojongerius/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ojongerius/subscriptions","organizations_url":"https://api.github.com/users/ojongerius/orgs","repos_url":"https://api.github.com/users/ojongerius/repos","events_url":"https://api.github.com/users/ojongerius/events{/privacy}","received_events_url":"https://api.github.com/users/ojongerius/received_events","type":"User","site_admin":false}],"milestone":null,"comments":9,"created_at":"2017-11-02T23:08:12Z","updated_at":"2017-11-04T11:12:43Z","closed_at":null,"author_association":"OWNER","pull_request":{"url":"https://api.github.com/repos/volCommunity/vol-django/pulls/86","html_url":"https://github.com/volCommunity/vol-django/pull/86","diff_url":"https://github.com/volCommunity/vol-django/pull/86.diff","patch_url":"https://github.com/volCommunity/vol-django/pull/86.patch"},"body":"Note that this PR includes database migrations to create unique UUIDs for all existing objects.\r\nNotable changes are:\r\n\r\n* Add some healthy data abstraction by exposing UUIDs instead of IDs\r\n* Support nested resource creation.\r\n\r\nTo do:\r\n- [x] Add validators and corresponding tests for `update()`.\r\n- [x] Update documentation noting the limitation of updating child resources (organisation, sites and labels). See http://www.django-rest-framework.org/topics/documenting-your-api . The documentation on issue creation in the Github API nails it notes the section on labels during PUT. [[1](https://developer.github.com/v3/issues/#create-an-issue)]\r\n- [ ] Clean up: Implement TODOs in serialisers.py (mostly re-using validation code and improving validation error feedback).\r\n- [ ] Verify if PATCH is implemented correctly, and update doco for it.\r\n- [ ] Update crawler pipeline to make use of changes, most of the logic in the pipeline can now go\r\n- [ ] Close off #87.\r\n\r\nThis solves part of #80 and will make the crawler pipelines much more simple."},"comment":{"url":"https://api.github.com/repos/volCommunity/vol-django/issues/comments/341888875","html_url":"https://github.com/volCommunity/vol-django/pull/86#issuecomment-341888875","issue_url":"https://api.github.com/repos/volCommunity/vol-django/issues/86","id":341888875,"user":{"login":"coveralls","id":2354108,"avatar_url":"https://avatars1.githubusercontent.com/u/2354108?v=4","gravatar_id":"","url":"https://api.github.com/users/coveralls","html_url":"https://github.com/coveralls","followers_url":"https://api.github.com/users/coveralls/followers","following_url":"https://api.github.com/users/coveralls/following{/other_user}","gists_url":"https://api.github.com/users/coveralls/gists{/gist_id}","starred_url":"https://api.github.com/users/coveralls/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/coveralls/subscriptions","organizations_url":"https://api.github.com/users/coveralls/orgs","repos_url":"https://api.github.com/users/coveralls/repos","events_url":"https://api.github.com/users/coveralls/events{/privacy}","received_events_url":"https://api.github.com/users/coveralls/received_events","type":"User","site_admin":false},"created_at":"2017-11-04T11:12:43Z","updated_at":"2017-11-04T11:12:43Z","author_association":"NONE","body":"\n[![Coverage Status](https://coveralls.io/builds/14037819/badge)](https://coveralls.io/builds/14037819)\n\nCoverage increased (+0.05%) to 99.318% when pulling **041a85d06406dcbf19d895ef17b1a1dc6dc21497 on uuids_nested_job** into **805ae3345daaf97a5d00ecd977c06a5560b50336 on master**.\n"}}
{"actor":{"display_login":"coveralls"}}