{"action":"closed","issue":{"url":"https://api.github.com/repos/bounswe/bounswe2016group11/issues/24","repository_url":"https://api.github.com/repos/bounswe/bounswe2016group11","labels_url":"https://api.github.com/repos/bounswe/bounswe2016group11/issues/24/labels{/name}","comments_url":"https://api.github.com/repos/bounswe/bounswe2016group11/issues/24/comments","events_url":"https://api.github.com/repos/bounswe/bounswe2016group11/issues/24/events","html_url":"https://github.com/bounswe/bounswe2016group11/issues/24","id":146264665,"number":24,"title":"Assignment 5 - Peer Review of Group 4","user":{"login":"bozis","id":17177714,"avatar_url":"https://avatars.githubusercontent.com/u/17177714?v=3","gravatar_id":"","url":"https://api.github.com/users/bozis","html_url":"https://github.com/bozis","followers_url":"https://api.github.com/users/bozis/followers","following_url":"https://api.github.com/users/bozis/following{/other_user}","gists_url":"https://api.github.com/users/bozis/gists{/gist_id}","starred_url":"https://api.github.com/users/bozis/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bozis/subscriptions","organizations_url":"https://api.github.com/users/bozis/orgs","repos_url":"https://api.github.com/users/bozis/repos","events_url":"https://api.github.com/users/bozis/events{/privacy}","received_events_url":"https://api.github.com/users/bozis/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2016-04-06T10:17:10Z","updated_at":"2016-05-14T18:41:49Z","closed_at":"2016-05-14T18:41:49Z","body":"**Use Cases**\r\nUse Cases seem to have been prepared with care. It can be easily understood what each user case defines.\r\n\r\n**Sequence Diagrams**\r\nI think there are some minor mistakes about drawings in sequence diagram in semantic tag. When they are corrected, it will look better. Lifecycles of  Page Manager and Tag Manager should be adjusted so as to include the functions they are related. Other than that, sequence diagrams are good.\r\n\r\n**Activity Diagram**\r\nWhen I opened the activity diagram at first, it scared me a little bit. But after some inspection, I found it very comprehensive. It is clearly understood that members of group 4 put a distinguishable labor in it. I will make only one suggestion here. For easy inspection, the diagram would be divided into subdiagrams. In this way, the diagram would look less intimidating :)\r\n\r\n**Class Diagram**\r\nClass functions and fields of class diagram are just fine. But there is one point that made me think. I guess arrows among classes are not suitable to the definition of \"class diagram\". I mean they do not sufficiently define the relationships between classes. They do not show for example the type of relationship is for example many-to-one or one-to-one. With the improvement of arrows, class diagram would look better. "}}