{"action":"created","issue":{"url":"https://api.github.com/repos/adlnet/xapi-lab/issues/10","labels_url":"https://api.github.com/repos/adlnet/xapi-lab/issues/10/labels{/name}","comments_url":"https://api.github.com/repos/adlnet/xapi-lab/issues/10/comments","events_url":"https://api.github.com/repos/adlnet/xapi-lab/issues/10/events","html_url":"https://github.com/adlnet/xapi-lab/issues/10","id":59123039,"number":10,"title":"My comments","user":{"login":"garemoko","id":1363163,"avatar_url":"https://avatars.githubusercontent.com/u/1363163?v=3","gravatar_id":"","url":"https://api.github.com/users/garemoko","html_url":"https://github.com/garemoko","followers_url":"https://api.github.com/users/garemoko/followers","following_url":"https://api.github.com/users/garemoko/following{/other_user}","gists_url":"https://api.github.com/users/garemoko/gists{/gist_id}","starred_url":"https://api.github.com/users/garemoko/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/garemoko/subscriptions","organizations_url":"https://api.github.com/users/garemoko/orgs","repos_url":"https://api.github.com/users/garemoko/repos","events_url":"https://api.github.com/users/garemoko/events{/privacy}","received_events_url":"https://api.github.com/users/garemoko/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/adlnet/xapi-lab/labels/feedback","name":"feedback","color":"d4c5f9"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-02-26T19:00:08Z","updated_at":"2015-02-26T22:03:38Z","closed_at":null,"body":"Hi! \r\nI'm planning on testing this more closely, but initial impression is that this is a really great tool. Here's some feedback from me on how I think it could be improved:\r\n\r\n* On the statement viewer, move the \"LRS Basic Auth Setup\" to the top of the page to match the Lab. Include the test functionality in the Lab.\r\n* Try to reduce the amount of direct json entry, perhaps copying the approach (or even code) used here: http://garemoko.github.io/tinStatement/tinstatement.htm I do really like that any field is possible using json as a backup though. \r\n* If you enter invalid json into the sub statement box, the tool corrects it when sending the statement and doesn't show me an error. \r\n* Statement result and context are missing from my statement. I didn't narrow down why or if that's always the case. \r\n* Statement id is listed as part of context, but it's not part of context, it's a property of the statement. \r\n\r\nI hope this feedback is helpful!"},"comment":{"url":"https://api.github.com/repos/adlnet/xapi-lab/issues/comments/76284166","html_url":"https://github.com/adlnet/xapi-lab/issues/10#issuecomment-76284166","issue_url":"https://api.github.com/repos/adlnet/xapi-lab/issues/10","id":76284166,"user":{"login":"garemoko","id":1363163,"avatar_url":"https://avatars.githubusercontent.com/u/1363163?v=3","gravatar_id":"","url":"https://api.github.com/users/garemoko","html_url":"https://github.com/garemoko","followers_url":"https://api.github.com/users/garemoko/followers","following_url":"https://api.github.com/users/garemoko/following{/other_user}","gists_url":"https://api.github.com/users/garemoko/gists{/gist_id}","starred_url":"https://api.github.com/users/garemoko/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/garemoko/subscriptions","organizations_url":"https://api.github.com/users/garemoko/orgs","repos_url":"https://api.github.com/users/garemoko/repos","events_url":"https://api.github.com/users/garemoko/events{/privacy}","received_events_url":"https://api.github.com/users/garemoko/received_events","type":"User","site_admin":false},"created_at":"2015-02-26T22:03:38Z","updated_at":"2015-02-26T22:03:38Z","body":"I'll certainly try and make some time to replicate it and document, yes. "}}