{"action":"closed","issue":{"url":"https://api.github.com/repos/CS2103JAN2017-T16-B1/main/issues/63","repository_url":"https://api.github.com/repos/CS2103JAN2017-T16-B1/main","labels_url":"https://api.github.com/repos/CS2103JAN2017-T16-B1/main/issues/63/labels{/name}","comments_url":"https://api.github.com/repos/CS2103JAN2017-T16-B1/main/issues/63/comments","events_url":"https://api.github.com/repos/CS2103JAN2017-T16-B1/main/issues/63/events","html_url":"https://github.com/CS2103JAN2017-T16-B1/main/issues/63","id":217210841,"number":63,"title":"V0.3 feedback","user":{"login":"brandonyeoxg","id":16134861,"avatar_url":"https://avatars0.githubusercontent.com/u/16134861?v=3","gravatar_id":"","url":"https://api.github.com/users/brandonyeoxg","html_url":"https://github.com/brandonyeoxg","followers_url":"https://api.github.com/users/brandonyeoxg/followers","following_url":"https://api.github.com/users/brandonyeoxg/following{/other_user}","gists_url":"https://api.github.com/users/brandonyeoxg/gists{/gist_id}","starred_url":"https://api.github.com/users/brandonyeoxg/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/brandonyeoxg/subscriptions","organizations_url":"https://api.github.com/users/brandonyeoxg/orgs","repos_url":"https://api.github.com/users/brandonyeoxg/repos","events_url":"https://api.github.com/users/brandonyeoxg/events{/privacy}","received_events_url":"https://api.github.com/users/brandonyeoxg/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2017-03-27T11:06:13Z","updated_at":"2017-03-30T08:21:08Z","closed_at":"2017-03-30T08:21:08Z","body":"**User Guide**\r\nPerhaps add more partial screenshots that captures the usage of command(s)?\r\n\r\nThe 'Command summary' section will be used heavily by our testers. Perhaps representing it in a table will make it clearer? It is good that examples have been added to the command summary =).\r\n\r\n**Developer Guide**\r\nDo remember to update the diagrams(sequence diagram, etc) to reflect task manager and not addressbook.\r\n\r\n![dg_person](https://cloud.githubusercontent.com/assets/16134861/24353473/49eb0a5e-1320-11e7-8a3f-305c9c4f775f.PNG)\r\n\r\n_Glossary_\r\nPerhaps definitons of what is a floating task, etc can be included?\r\n\r\n_Product Survey_\r\nGlad to see extensive product survey was done =).\r\nPerhaps it would be clearer to display it in a table?\r\n\r\nIt may be more practical to give a few examples of interesting test cases and explain the thinking behind how you arrived at those test cases, rather than try to give a comprehensive list of all test cases.\r\n\r\nPerhaps explain how your design support undo/redo. There are multiple ways this can be done, so it will be useful to explain how you do it and why you chose to do it that way.\r\n\r\n**Overall**\r\nTry to do a peer review on each other's PRs.\r\nDo make sure to pass your tests too.\r\nDo make changes to `AboutUs.md` to only have your team members' picture, remove the source code team's pictures.\r\nPlease close this issue after reading it."}}
{"actor":{"display_login":"kailii"}}