{"action":"closed","issue":{"url":"https://api.github.com/repos/kristofferlind/1dv450-project-kl222jy/issues/2","labels_url":"https://api.github.com/repos/kristofferlind/1dv450-project-kl222jy/issues/2/labels{/name}","comments_url":"https://api.github.com/repos/kristofferlind/1dv450-project-kl222jy/issues/2/comments","events_url":"https://api.github.com/repos/kristofferlind/1dv450-project-kl222jy/issues/2/events","html_url":"https://github.com/kristofferlind/1dv450-project-kl222jy/issues/2","id":57461837,"number":2,"title":"Peer Review","user":{"login":"HenrikGabrielsson","id":5389712,"avatar_url":"https://avatars.githubusercontent.com/u/5389712?v=3","gravatar_id":"","url":"https://api.github.com/users/HenrikGabrielsson","html_url":"https://github.com/HenrikGabrielsson","followers_url":"https://api.github.com/users/HenrikGabrielsson/followers","following_url":"https://api.github.com/users/HenrikGabrielsson/following{/other_user}","gists_url":"https://api.github.com/users/HenrikGabrielsson/gists{/gist_id}","starred_url":"https://api.github.com/users/HenrikGabrielsson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/HenrikGabrielsson/subscriptions","organizations_url":"https://api.github.com/users/HenrikGabrielsson/orgs","repos_url":"https://api.github.com/users/HenrikGabrielsson/repos","events_url":"https://api.github.com/users/HenrikGabrielsson/events{/privacy}","received_events_url":"https://api.github.com/users/HenrikGabrielsson/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-02-12T13:46:44Z","updated_at":"2015-02-19T11:32:41Z","closed_at":"2015-02-19T11:32:41Z","body":"Hej!\r\n\r\nHär kommer lite kritik på din applikation. Bli inte ledsen.\r\n\r\n# Installation\r\nInstallationen var väldigt enkel och bra beskriven. Inga problem alls.\r\n\r\n# Felhantering\r\nMan får bra, beskrivande feedback från applikationen när man utför olika handlingar. Men det hade kanske varit bra med feedback eller en felsida när man till exempel försöker komma åt en nyckel som inte finns eller när man får en 404.\r\n\r\n# Validering\r\nBra validering, förutom email-adressen som tillåter allt som har ett @ i sig.\r\n\r\n# Säkerhet\r\nJag kan inte hitta några säkerhetshål, så det verkar OK ;)\r\n\r\n# Kodkvalitet\r\n\r\nKoden innehåller väldigt lite kommentarer, vilket hade hjälpt att förstå koden, men det kanske inte behövs eftersom koden ändå är ganska enkel att förstå. Inte mycket repetition. Bra! \r\n\r\n# Gränssnitt/Funktionalitet\r\nAdmin-gränssnittet är mycket snyggt och enkelt att använda. Mycket funktionalitet, och allt verkar fungera. Som användare hade det kanske varit bra om knappen “Remove key” inte fanns kvar när nyckeln redan är borttagen. Detta borde gå och fixa med en enkel if-sats.\r\n\r\n\r\nSmåfel, men i stort sätt tycker jag att allt fungerar bra och säkert. MVG++\r\n\r\nMVH Henrik Gabrielsson, hg222aa"}}