{"action":"created","comment":{"url":"https://api.github.com/repos/LibraryOfCongress/chronam/pulls/comments/148550590","pull_request_review_id":73809492,"id":148550590,"diff_hunk":"@@ -23,6 +23,7 @@\n from chronam.core.utils.url import unpack_url_path\n from chronam.core import models, index\n from chronam.core.rdf import title_to_graph, issue_to_graph, page_to_graph\n+from chronam.core.index import get_page_text","path":"core/views/browse.py","position":4,"original_position":4,"commit_id":"60913dc996ef09e510fd7ea3f475b1ebb8ffa6c6","original_commit_id":"60913dc996ef09e510fd7ea3f475b1ebb8ffa6c6","user":{"login":"acdha","id":46565,"avatar_url":"https://avatars3.githubusercontent.com/u/46565?v=4","gravatar_id":"","url":"https://api.github.com/users/acdha","html_url":"https://github.com/acdha","followers_url":"https://api.github.com/users/acdha/followers","following_url":"https://api.github.com/users/acdha/following{/other_user}","gists_url":"https://api.github.com/users/acdha/gists{/gist_id}","starred_url":"https://api.github.com/users/acdha/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/acdha/subscriptions","organizations_url":"https://api.github.com/users/acdha/orgs","repos_url":"https://api.github.com/users/acdha/repos","events_url":"https://api.github.com/users/acdha/events{/privacy}","received_events_url":"https://api.github.com/users/acdha/received_events","type":"User","site_admin":false},"body":"Nothing wrong with this import but it would be nice to have them sorted using something like `isort` for readability.","created_at":"2017-11-02T14:35:26Z","updated_at":"2017-11-02T14:35:26Z","html_url":"https://github.com/LibraryOfCongress/chronam/pull/159#discussion_r148550590","pull_request_url":"https://api.github.com/repos/LibraryOfCongress/chronam/pulls/159","author_association":"MEMBER","_links":{"self":{"href":"https://api.github.com/repos/LibraryOfCongress/chronam/pulls/comments/148550590"},"html":{"href":"https://github.com/LibraryOfCongress/chronam/pull/159#discussion_r148550590"},"pull_request":{"href":"https://api.github.com/repos/LibraryOfCongress/chronam/pulls/159"}}},"pull_request":{"url":"https://api.github.com/repos/LibraryOfCongress/chronam/pulls/159","id":149274681,"html_url":"https://github.com/LibraryOfCongress/chronam/pull/159","diff_url":"https://github.com/LibraryOfCongress/chronam/pull/159.diff","patch_url":"https://github.com/LibraryOfCongress/chronam/pull/159.patch","issue_url":"https://api.github.com/repos/LibraryOfCongress/chronam/issues/159","number":159,"state":"open","locked":false,"title":"Chronam 344, 345, 346","user":{"login":"lsukharn","id":6530372,"avatar_url":"https://avatars0.githubusercontent.com/u/6530372?v=4","gravatar_id":"","url":"https://api.github.com/users/lsukharn","html_url":"https://github.com/lsukharn","followers_url":"https://api.github.com/users/lsukharn/followers","following_url":"https://api.github.com/users/lsukharn/following{/other_user}","gists_url":"https://api.github.com/users/lsukharn/gists{/gist_id}","starred_url":"https://api.github.com/users/lsukharn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lsukharn/subscriptions","organizations_url":"https://api.github.com/users/lsukharn/orgs","repos_url":"https://api.github.com/users/lsukharn/repos","events_url":"https://api.github.com/users/lsukharn/events{/privacy}","received_events_url":"https://api.github.com/users/lsukharn/received_events","type":"User","site_admin":false},"body":"Please review @myusuf  @johnscancella \r\ntickets: \r\nchronam-344(move get_text_from_solr to index.py),\r\nchronam-345(add new unit/integration tests),\r\nchronam-346(drop text field from db) and\r\nchronam-357(modifying batch_loader.py for new db schema)","created_at":"2017-10-27T20:04:07Z","updated_at":"2017-11-02T14:35:26Z","closed_at":null,"merged_at":null,"merge_commit_sha":"4562b71020908ed82f0e2d9566a458630f8d761f","assignee":null,"assignees":[],"requested_reviewers":[],"milestone":null,"commits_url":"https://api.github.com/repos/LibraryOfCongress/chronam/pulls/159/commits","review_comments_url":"https://api.github.com/repos/LibraryOfCongress/chronam/pulls/159/comments","review_comment_url":"https://api.github.com/repos/LibraryOfCongress/chronam/pulls/comments{/number}","comments_url":"https://api.github.com/repos/LibraryOfCongress/chronam/issues/159/comments","statuses_url":"https://api.github.com/repos/LibraryOfCongress/chronam/statuses/60913dc996ef09e510fd7ea3f475b1ebb8ffa6c6","head":{"label":"lsukharn:chronam_345","ref":"chronam_345","sha":"60913dc996ef09e510fd7ea3f475b1ebb8ffa6c6","user":{"login":"lsukharn","id":6530372,"avatar_url":"https://avatars0.githubusercontent.com/u/6530372?v=4","gravatar_id":"","url":"https://api.github.com/users/lsukharn","html_url":"https://github.com/lsukharn","followers_url":"https://api.github.com/users/lsukharn/followers","following_url":"https://api.github.com/users/lsukharn/following{/other_user}","gists_url":"https://api.github.com/users/lsukharn/gists{/gist_id}","starred_url":"https://api.github.com/users/lsukharn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lsukharn/subscriptions","organizations_url":"https://api.github.com/users/lsukharn/orgs","repos_url":"https://api.github.com/users/lsukharn/repos","events_url":"https://api.github.com/users/lsukharn/events{/privacy}","received_events_url":"https://api.github.com/users/lsukharn/received_events","type":"User","site_admin":false},"repo":{"id":106858893,"name":"chronam","full_name":"lsukharn/chronam","owner":{"login":"lsukharn","id":6530372,"avatar_url":"https://avatars0.githubusercontent.com/u/6530372?v=4","gravatar_id":"","url":"https://api.github.com/users/lsukharn","html_url":"https://github.com/lsukharn","followers_url":"https://api.github.com/users/lsukharn/followers","following_url":"https://api.github.com/users/lsukharn/following{/other_user}","gists_url":"https://api.github.com/users/lsukharn/gists{/gist_id}","starred_url":"https://api.github.com/users/lsukharn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lsukharn/subscriptions","organizations_url":"https://api.github.com/users/lsukharn/orgs","repos_url":"https://api.github.com/users/lsukharn/repos","events_url":"https://api.github.com/users/lsukharn/events{/privacy}","received_events_url":"https://api.github.com/users/lsukharn/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/lsukharn/chronam","description":"A project of the Library of Congress. Note: project members may work on both official Library of Congress projects and non-LC projects.  Project mailing list can be found at http://listserv.loc.gov/archives/chronam-users.html.","fork":true,"url":"https://api.github.com/repos/lsukharn/chronam","forks_url":"https://api.github.com/repos/lsukharn/chronam/forks","keys_url":"https://api.github.com/repos/lsukharn/chronam/keys{/key_id}","collaborators_url":"https://api.github.com/repos/lsukharn/chronam/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/lsukharn/chronam/teams","hooks_url":"https://api.github.com/repos/lsukharn/chronam/hooks","issue_events_url":"https://api.github.com/repos/lsukharn/chronam/issues/events{/number}","events_url":"https://api.github.com/repos/lsukharn/chronam/events","assignees_url":"https://api.github.com/repos/lsukharn/chronam/assignees{/user}","branches_url":"https://api.github.com/repos/lsukharn/chronam/branches{/branch}","tags_url":"https://api.github.com/repos/lsukharn/chronam/tags","blobs_url":"https://api.github.com/repos/lsukharn/chronam/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/lsukharn/chronam/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/lsukharn/chronam/git/refs{/sha}","trees_url":"https://api.github.com/repos/lsukharn/chronam/git/trees{/sha}","statuses_url":"https://api.github.com/repos/lsukharn/chronam/statuses/{sha}","languages_url":"https://api.github.com/repos/lsukharn/chronam/languages","stargazers_url":"https://api.github.com/repos/lsukharn/chronam/stargazers","contributors_url":"https://api.github.com/repos/lsukharn/chronam/contributors","subscribers_url":"https://api.github.com/repos/lsukharn/chronam/subscribers","subscription_url":"https://api.github.com/repos/lsukharn/chronam/subscription","commits_url":"https://api.github.com/repos/lsukharn/chronam/commits{/sha}","git_commits_url":"https://api.github.com/repos/lsukharn/chronam/git/commits{/sha}","comments_url":"https://api.github.com/repos/lsukharn/chronam/comments{/number}","issue_comment_url":"https://api.github.com/repos/lsukharn/chronam/issues/comments{/number}","contents_url":"https://api.github.com/repos/lsukharn/chronam/contents/{+path}","compare_url":"https://api.github.com/repos/lsukharn/chronam/compare/{base}...{head}","merges_url":"https://api.github.com/repos/lsukharn/chronam/merges","archive_url":"https://api.github.com/repos/lsukharn/chronam/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/lsukharn/chronam/downloads","issues_url":"https://api.github.com/repos/lsukharn/chronam/issues{/number}","pulls_url":"https://api.github.com/repos/lsukharn/chronam/pulls{/number}","milestones_url":"https://api.github.com/repos/lsukharn/chronam/milestones{/number}","notifications_url":"https://api.github.com/repos/lsukharn/chronam/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/lsukharn/chronam/labels{/name}","releases_url":"https://api.github.com/repos/lsukharn/chronam/releases{/id}","deployments_url":"https://api.github.com/repos/lsukharn/chronam/deployments","created_at":"2017-10-13T18:20:27Z","updated_at":"2017-10-13T18:20:30Z","pushed_at":"2017-11-01T20:35:42Z","git_url":"git://github.com/lsukharn/chronam.git","ssh_url":"git@github.com:lsukharn/chronam.git","clone_url":"https://github.com/lsukharn/chronam.git","svn_url":"https://github.com/lsukharn/chronam","homepage":"","size":9067,"stargazers_count":0,"watchers_count":0,"language":"Python","has_issues":false,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"archived":false,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"LibraryOfCongress:master","ref":"master","sha":"7ccecd20d8786fbe35cfdce70ec02b8b8d9a1a69","user":{"login":"LibraryOfCongress","id":2070730,"avatar_url":"https://avatars3.githubusercontent.com/u/2070730?v=4","gravatar_id":"","url":"https://api.github.com/users/LibraryOfCongress","html_url":"https://github.com/LibraryOfCongress","followers_url":"https://api.github.com/users/LibraryOfCongress/followers","following_url":"https://api.github.com/users/LibraryOfCongress/following{/other_user}","gists_url":"https://api.github.com/users/LibraryOfCongress/gists{/gist_id}","starred_url":"https://api.github.com/users/LibraryOfCongress/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/LibraryOfCongress/subscriptions","organizations_url":"https://api.github.com/users/LibraryOfCongress/orgs","repos_url":"https://api.github.com/users/LibraryOfCongress/repos","events_url":"https://api.github.com/users/LibraryOfCongress/events{/privacy}","received_events_url":"https://api.github.com/users/LibraryOfCongress/received_events","type":"Organization","site_admin":false},"repo":{"id":7136938,"name":"chronam","full_name":"LibraryOfCongress/chronam","owner":{"login":"LibraryOfCongress","id":2070730,"avatar_url":"https://avatars3.githubusercontent.com/u/2070730?v=4","gravatar_id":"","url":"https://api.github.com/users/LibraryOfCongress","html_url":"https://github.com/LibraryOfCongress","followers_url":"https://api.github.com/users/LibraryOfCongress/followers","following_url":"https://api.github.com/users/LibraryOfCongress/following{/other_user}","gists_url":"https://api.github.com/users/LibraryOfCongress/gists{/gist_id}","starred_url":"https://api.github.com/users/LibraryOfCongress/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/LibraryOfCongress/subscriptions","organizations_url":"https://api.github.com/users/LibraryOfCongress/orgs","repos_url":"https://api.github.com/users/LibraryOfCongress/repos","events_url":"https://api.github.com/users/LibraryOfCongress/events{/privacy}","received_events_url":"https://api.github.com/users/LibraryOfCongress/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/LibraryOfCongress/chronam","description":"A project of the Library of Congress. Note: project members may work on both official Library of Congress projects and non-LC projects.  Project mailing list can be found at http://listserv.loc.gov/archives/chronam-users.html.","fork":false,"url":"https://api.github.com/repos/LibraryOfCongress/chronam","forks_url":"https://api.github.com/repos/LibraryOfCongress/chronam/forks","keys_url":"https://api.github.com/repos/LibraryOfCongress/chronam/keys{/key_id}","collaborators_url":"https://api.github.com/repos/LibraryOfCongress/chronam/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/LibraryOfCongress/chronam/teams","hooks_url":"https://api.github.com/repos/LibraryOfCongress/chronam/hooks","issue_events_url":"https://api.github.com/repos/LibraryOfCongress/chronam/issues/events{/number}","events_url":"https://api.github.com/repos/LibraryOfCongress/chronam/events","assignees_url":"https://api.github.com/repos/LibraryOfCongress/chronam/assignees{/user}","branches_url":"https://api.github.com/repos/LibraryOfCongress/chronam/branches{/branch}","tags_url":"https://api.github.com/repos/LibraryOfCongress/chronam/tags","blobs_url":"https://api.github.com/repos/LibraryOfCongress/chronam/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/LibraryOfCongress/chronam/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/LibraryOfCongress/chronam/git/refs{/sha}","trees_url":"https://api.github.com/repos/LibraryOfCongress/chronam/git/trees{/sha}","statuses_url":"https://api.github.com/repos/LibraryOfCongress/chronam/statuses/{sha}","languages_url":"https://api.github.com/repos/LibraryOfCongress/chronam/languages","stargazers_url":"https://api.github.com/repos/LibraryOfCongress/chronam/stargazers","contributors_url":"https://api.github.com/repos/LibraryOfCongress/chronam/contributors","subscribers_url":"https://api.github.com/repos/LibraryOfCongress/chronam/subscribers","subscription_url":"https://api.github.com/repos/LibraryOfCongress/chronam/subscription","commits_url":"https://api.github.com/repos/LibraryOfCongress/chronam/commits{/sha}","git_commits_url":"https://api.github.com/repos/LibraryOfCongress/chronam/git/commits{/sha}","comments_url":"https://api.github.com/repos/LibraryOfCongress/chronam/comments{/number}","issue_comment_url":"https://api.github.com/repos/LibraryOfCongress/chronam/issues/comments{/number}","contents_url":"https://api.github.com/repos/LibraryOfCongress/chronam/contents/{+path}","compare_url":"https://api.github.com/repos/LibraryOfCongress/chronam/compare/{base}...{head}","merges_url":"https://api.github.com/repos/LibraryOfCongress/chronam/merges","archive_url":"https://api.github.com/repos/LibraryOfCongress/chronam/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/LibraryOfCongress/chronam/downloads","issues_url":"https://api.github.com/repos/LibraryOfCongress/chronam/issues{/number}","pulls_url":"https://api.github.com/repos/LibraryOfCongress/chronam/pulls{/number}","milestones_url":"https://api.github.com/repos/LibraryOfCongress/chronam/milestones{/number}","notifications_url":"https://api.github.com/repos/LibraryOfCongress/chronam/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/LibraryOfCongress/chronam/labels{/name}","releases_url":"https://api.github.com/repos/LibraryOfCongress/chronam/releases{/id}","deployments_url":"https://api.github.com/repos/LibraryOfCongress/chronam/deployments","created_at":"2012-12-12T20:11:42Z","updated_at":"2017-07-22T19:20:22Z","pushed_at":"2017-11-01T20:35:43Z","git_url":"git://github.com/LibraryOfCongress/chronam.git","ssh_url":"git@github.com:LibraryOfCongress/chronam.git","clone_url":"https://github.com/LibraryOfCongress/chronam.git","svn_url":"https://github.com/LibraryOfCongress/chronam","homepage":"","size":9027,"stargazers_count":49,"watchers_count":49,"language":"Python","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":25,"mirror_url":null,"archived":false,"open_issues_count":53,"forks":25,"open_issues":53,"watchers":49,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/LibraryOfCongress/chronam/pulls/159"},"html":{"href":"https://github.com/LibraryOfCongress/chronam/pull/159"},"issue":{"href":"https://api.github.com/repos/LibraryOfCongress/chronam/issues/159"},"comments":{"href":"https://api.github.com/repos/LibraryOfCongress/chronam/issues/159/comments"},"review_comments":{"href":"https://api.github.com/repos/LibraryOfCongress/chronam/pulls/159/comments"},"review_comment":{"href":"https://api.github.com/repos/LibraryOfCongress/chronam/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/LibraryOfCongress/chronam/pulls/159/commits"},"statuses":{"href":"https://api.github.com/repos/LibraryOfCongress/chronam/statuses/60913dc996ef09e510fd7ea3f475b1ebb8ffa6c6"}},"author_association":"NONE"}}
{"actor":{"display_login":"acdha"}}