{"action":"created","issue":{"url":"https://api.github.com/repos/mitodl/lore/issues/9","labels_url":"https://api.github.com/repos/mitodl/lore/issues/9/labels{/name}","comments_url":"https://api.github.com/repos/mitodl/lore/issues/9/comments","events_url":"https://api.github.com/repos/mitodl/lore/issues/9/events","html_url":"https://github.com/mitodl/lore/pull/9","id":76035872,"number":9,"title":"Add taxonomy app with models","user":{"login":"noisecapella","id":863262,"avatar_url":"https://avatars.githubusercontent.com/u/863262?v=3","gravatar_id":"","url":"https://api.github.com/users/noisecapella","html_url":"https://github.com/noisecapella","followers_url":"https://api.github.com/users/noisecapella/followers","following_url":"https://api.github.com/users/noisecapella/following{/other_user}","gists_url":"https://api.github.com/users/noisecapella/gists{/gist_id}","starred_url":"https://api.github.com/users/noisecapella/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/noisecapella/subscriptions","organizations_url":"https://api.github.com/users/noisecapella/orgs","repos_url":"https://api.github.com/users/noisecapella/repos","events_url":"https://api.github.com/users/noisecapella/events{/privacy}","received_events_url":"https://api.github.com/users/noisecapella/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/mitodl/lore/labels/work%20in%20progress","name":"work in progress","color":"fad8c7"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":11,"created_at":"2015-05-13T16:02:37Z","updated_at":"2015-05-14T21:13:26Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/mitodl/lore/pulls/9","html_url":"https://github.com/mitodl/lore/pull/9","diff_url":"https://github.com/mitodl/lore/pull/9.diff","patch_url":"https://github.com/mitodl/lore/pull/9.patch"},"body":"This isn't done, there are no API methods yet, but how does this look from a Django perspective? Is everything in the right place? Any other comments on the model?"},"comment":{"url":"https://api.github.com/repos/mitodl/lore/issues/comments/102170916","html_url":"https://github.com/mitodl/lore/pull/9#issuecomment-102170916","issue_url":"https://api.github.com/repos/mitodl/lore/issues/9","id":102170916,"user":{"login":"noisecapella","id":863262,"avatar_url":"https://avatars.githubusercontent.com/u/863262?v=3","gravatar_id":"","url":"https://api.github.com/users/noisecapella","html_url":"https://github.com/noisecapella","followers_url":"https://api.github.com/users/noisecapella/followers","following_url":"https://api.github.com/users/noisecapella/following{/other_user}","gists_url":"https://api.github.com/users/noisecapella/gists{/gist_id}","starred_url":"https://api.github.com/users/noisecapella/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/noisecapella/subscriptions","organizations_url":"https://api.github.com/users/noisecapella/orgs","repos_url":"https://api.github.com/users/noisecapella/repos","events_url":"https://api.github.com/users/noisecapella/events{/privacy}","received_events_url":"https://api.github.com/users/noisecapella/received_events","type":"User","site_admin":false},"created_at":"2015-05-14T21:13:26Z","updated_at":"2015-05-14T21:13:26Z","body":"Regarding this code snippet:\r\n\r\n    def add_learning_object_type_for_vocabulary(learning_object_type, vocabulary_id):\r\n        vocabulary = Vocabulary.objects.get(id=vocabulary_id)\r\n        learning_object_type = LearningObjectType.objects.get(name=learning_object_type)\r\n        vocabulary.learning_object_types.add(learning_object_type)\r\n\r\nThe two `get()`s don't seem to be required since the SQL is just inserting a row with two integer ids for the m2m table. Is there another Django-friendly way to do this? This may also be premature optimization, just wondering"}}