{"action":"created","issue":{"url":"https://api.github.com/repos/ResearchStack/ResearchStack/issues/246","repository_url":"https://api.github.com/repos/ResearchStack/ResearchStack","labels_url":"https://api.github.com/repos/ResearchStack/ResearchStack/issues/246/labels{/name}","comments_url":"https://api.github.com/repos/ResearchStack/ResearchStack/issues/246/comments","events_url":"https://api.github.com/repos/ResearchStack/ResearchStack/issues/246/events","html_url":"https://github.com/ResearchStack/ResearchStack/pull/246","id":154599532,"number":246,"title":"added code to skin/SmartSurvey.java to support text-only steps in surveys","user":{"login":"crayne","id":134657,"avatar_url":"https://avatars.githubusercontent.com/u/134657?v=3","gravatar_id":"","url":"https://api.github.com/users/crayne","html_url":"https://github.com/crayne","followers_url":"https://api.github.com/users/crayne/followers","following_url":"https://api.github.com/users/crayne/following{/other_user}","gists_url":"https://api.github.com/users/crayne/gists{/gist_id}","starred_url":"https://api.github.com/users/crayne/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/crayne/subscriptions","organizations_url":"https://api.github.com/users/crayne/orgs","repos_url":"https://api.github.com/users/crayne/repos","events_url":"https://api.github.com/users/crayne/events{/privacy}","received_events_url":"https://api.github.com/users/crayne/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":7,"created_at":"2016-05-12T22:56:10Z","updated_at":"2016-05-28T16:31:52Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/ResearchStack/ResearchStack/pulls/246","html_url":"https://github.com/ResearchStack/ResearchStack/pull/246","diff_url":"https://github.com/ResearchStack/ResearchStack/pull/246.diff","patch_url":"https://github.com/ResearchStack/ResearchStack/pull/246.patch"},"body":"Please consider only code in skin/SmartSurvey.java.  Code is inserted which allows a user to create a survey step which consists of text only -- no question.  This can be specified in a JSON file by defining type as \"SurveyTextOnly\". Example:\r\n\r\n````\r\n      {\r\n         \"guid\":\"535d9b7e-6548-4bf8-8a4d-fb113211e684\",\r\n         \"identifier\":\"weightInitial\",\r\n         \"type\":\"SurveyTextOnly\",\r\n         \"prompt\":\"Consistent Time of Day\",\r\n          \"promptDetail\":\"It is best to weigh yourself at a regular hour, every time. A good time can be early in the morning before eating.\"\r\n      },\r\n````"},"comment":{"url":"https://api.github.com/repos/ResearchStack/ResearchStack/issues/comments/222317139","html_url":"https://github.com/ResearchStack/ResearchStack/pull/246#issuecomment-222317139","issue_url":"https://api.github.com/repos/ResearchStack/ResearchStack/issues/246","id":222317139,"user":{"login":"nrlakin","id":986783,"avatar_url":"https://avatars.githubusercontent.com/u/986783?v=3","gravatar_id":"","url":"https://api.github.com/users/nrlakin","html_url":"https://github.com/nrlakin","followers_url":"https://api.github.com/users/nrlakin/followers","following_url":"https://api.github.com/users/nrlakin/following{/other_user}","gists_url":"https://api.github.com/users/nrlakin/gists{/gist_id}","starred_url":"https://api.github.com/users/nrlakin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nrlakin/subscriptions","organizations_url":"https://api.github.com/users/nrlakin/orgs","repos_url":"https://api.github.com/users/nrlakin/repos","events_url":"https://api.github.com/users/nrlakin/events{/privacy}","received_events_url":"https://api.github.com/users/nrlakin/received_events","type":"User","site_admin":false},"created_at":"2016-05-28T16:31:52Z","updated_at":"2016-05-28T16:31:52Z","body":"@crayne I busted issued a PR with a working unit test in Skin, for DataProvider. With the current state of the master branch, tests should work if you simply add the test class to the right directory (skin/src/test/java...). I would recommend checking out the branch for this pull request, pulling from the upstream master (i.e., ResearchStack master), fixing any merge conflicts, than pushing back to the branch for this pull request. Merging with master should fix any dependency issues running the test and clean up the directory structure. If it works, the only files with changes will be:\r\n\r\n* skin/src/main/java/org/researchstack/skin/task/SmartSurveyTask.java\r\n* skin/src/test/java/org/researchstack/skin/task/SmartSurveyTask.java"}}
{"actor":{"display_login":"nrlakin"}}