{"action":"created","issue":{"url":"https://api.github.com/repos/MITLibraries/bento/issues/254","repository_url":"https://api.github.com/repos/MITLibraries/bento","labels_url":"https://api.github.com/repos/MITLibraries/bento/issues/254/labels{/name}","comments_url":"https://api.github.com/repos/MITLibraries/bento/issues/254/comments","events_url":"https://api.github.com/repos/MITLibraries/bento/issues/254/events","html_url":"https://github.com/MITLibraries/bento/pull/254","id":256356718,"number":254,"title":"514 fulltext link","user":{"login":"thatandromeda","id":959761,"avatar_url":"https://avatars2.githubusercontent.com/u/959761?v=4","gravatar_id":"","url":"https://api.github.com/users/thatandromeda","html_url":"https://github.com/thatandromeda","followers_url":"https://api.github.com/users/thatandromeda/followers","following_url":"https://api.github.com/users/thatandromeda/following{/other_user}","gists_url":"https://api.github.com/users/thatandromeda/gists{/gist_id}","starred_url":"https://api.github.com/users/thatandromeda/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/thatandromeda/subscriptions","organizations_url":"https://api.github.com/users/thatandromeda/orgs","repos_url":"https://api.github.com/users/thatandromeda/repos","events_url":"https://api.github.com/users/thatandromeda/events{/privacy}","received_events_url":"https://api.github.com/users/thatandromeda/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":12,"created_at":"2017-09-08T20:01:04Z","updated_at":"2017-09-15T20:35:30Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/MITLibraries/bento/pulls/254","html_url":"https://github.com/MITLibraries/bento/pull/254","diff_url":"https://github.com/MITLibraries/bento/pull/254.diff","patch_url":"https://github.com/MITLibraries/bento/pull/254.patch"},"body":"## Status\r\n**READY**\r\n\r\n~~Opening the PR to get the PR build, but I want UX to spend some quality time evaluating the EBSCO fulltext links.~~ UX feedback acquired; this is ready to roll.\r\n\r\n#### What does this PR do?\r\nAdds a link to online fulltext, where available, using the EBSCO gem's logic for prioritizing fulltext sources.\r\n\r\nRefactors `record.html.erb` and `RecordController` for readability.\r\n\r\n#### How can a reviewer manually see the effects of these changes?\r\n* Go to the PR build\r\n* Make sure to `/toggle/?feature=local_full_record`\r\n* Search for stuff\r\n* Look at its detail pages\r\n* Lo, a 'view online' button where available\r\n* Or a 'check for online copy' link if the top-choice link is SFX (see, e.g., https://mit-bento-staging-pr-254.herokuapp.com/record/edshtl/mdp.39015061775196)\r\n\r\n#### What are the relevant tickets?\r\n- https://mitlibraries.atlassian.net/browse/DI-514\r\n\r\n#### Screenshots (if appropriate)\r\n\r\n#### Todo:\r\n- [ ] Tests\r\n- [ ] Documentation\r\n- [x] Stakeholder approval\r\n\r\n#### Requires Database Migrations?\r\nNO\r\n\r\n#### Includes new or updated dependencies?\r\nNO\r\n"},"comment":{"url":"https://api.github.com/repos/MITLibraries/bento/issues/comments/329895018","html_url":"https://github.com/MITLibraries/bento/pull/254#issuecomment-329895018","issue_url":"https://api.github.com/repos/MITLibraries/bento/issues/254","id":329895018,"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-09-15T20:35:30Z","updated_at":"2017-09-15T20:35:30Z","author_association":"NONE","body":"\n[![Coverage Status](https://coveralls.io/builds/13294813/badge)](https://coveralls.io/builds/13294813)\n\nCoverage remained the same at 100.0% when pulling **9370185679f0d11432688947f9cb0450579da0b5 on 514_fulltext_link** into **d2b9afb829fe71a0f56c10066af077f444f5ab31 on master**.\n"}}
{"actor":{"display_login":"coveralls"}}