{"action":"created","issue":{"url":"https://api.github.com/repos/OpenClinica/OpenClinica/issues/195","labels_url":"https://api.github.com/repos/OpenClinica/OpenClinica/issues/195/labels{/name}","comments_url":"https://api.github.com/repos/OpenClinica/OpenClinica/issues/195/comments","events_url":"https://api.github.com/repos/OpenClinica/OpenClinica/issues/195/events","html_url":"https://github.com/OpenClinica/OpenClinica/pull/195","id":81839094,"number":195,"title":"Update signStudySubject.jsp","user":{"login":"iDrakos","id":10481651,"avatar_url":"https://avatars.githubusercontent.com/u/10481651?v=3","gravatar_id":"","url":"https://api.github.com/users/iDrakos","html_url":"https://github.com/iDrakos","followers_url":"https://api.github.com/users/iDrakos/followers","following_url":"https://api.github.com/users/iDrakos/following{/other_user}","gists_url":"https://api.github.com/users/iDrakos/gists{/gist_id}","starred_url":"https://api.github.com/users/iDrakos/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/iDrakos/subscriptions","organizations_url":"https://api.github.com/users/iDrakos/orgs","repos_url":"https://api.github.com/users/iDrakos/repos","events_url":"https://api.github.com/users/iDrakos/events{/privacy}","received_events_url":"https://api.github.com/users/iDrakos/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2015-05-28T09:07:49Z","updated_at":"2015-10-15T18:33:44Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/OpenClinica/OpenClinica/pulls/195","html_url":"https://github.com/OpenClinica/OpenClinica/pull/195","diff_url":"https://github.com/OpenClinica/OpenClinica/pull/195.diff","patch_url":"https://github.com/OpenClinica/OpenClinica/pull/195.patch"},"body":"On pull request #194 I’m proposing an update to the showSummaryBox js function to be able to handle both elements and elemants IDs.\r\n\r\nAlternative, if you prefer to keep global_functions_javascript.js intact, you can update the signStudySubject.jsp file and call the function showSummaryBox in the correct way (passing element ID, not the element).\r\n\r\nI am also suggesting that the discrepancy notes should be displayed by default, since the information that it contains is vital for many SOP’s. Therefor, link 436 should be changed from:\r\n \r\n    <div id=\"subjDiscNoteDiv\" class=\"subjDiscNoteDiv\" style=\"display:none\">\r\n\r\nto\r\n\r\n    <div id=\"subjDiscNoteDiv\" class=\"subjDiscNoteDiv\">"},"comment":{"url":"https://api.github.com/repos/OpenClinica/OpenClinica/issues/comments/148482458","html_url":"https://github.com/OpenClinica/OpenClinica/pull/195#issuecomment-148482458","issue_url":"https://api.github.com/repos/OpenClinica/OpenClinica/issues/195","id":148482458,"user":{"login":"angood","id":8268255,"avatar_url":"https://avatars.githubusercontent.com/u/8268255?v=3","gravatar_id":"","url":"https://api.github.com/users/angood","html_url":"https://github.com/angood","followers_url":"https://api.github.com/users/angood/followers","following_url":"https://api.github.com/users/angood/following{/other_user}","gists_url":"https://api.github.com/users/angood/gists{/gist_id}","starred_url":"https://api.github.com/users/angood/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/angood/subscriptions","organizations_url":"https://api.github.com/users/angood/orgs","repos_url":"https://api.github.com/users/angood/repos","events_url":"https://api.github.com/users/angood/events{/privacy}","received_events_url":"https://api.github.com/users/angood/received_events","type":"User","site_admin":false},"created_at":"2015-10-15T18:33:44Z","updated_at":"2015-10-15T18:33:44Z","body":"Hi @iDrakos  \r\nI agree that it is useful to have this information presented on the signature page as it's quite relevant to the data. We'll look into this and get back to you ASAP. \r\n\r\nThanks! \r\nAlicia"}}