{"action":"closed","issue":{"url":"https://api.github.com/repos/al223ec/1DV450-Registreringsapplikation/issues/1","labels_url":"https://api.github.com/repos/al223ec/1DV450-Registreringsapplikation/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/al223ec/1DV450-Registreringsapplikation/issues/1/comments","events_url":"https://api.github.com/repos/al223ec/1DV450-Registreringsapplikation/issues/1/events","html_url":"https://github.com/al223ec/1DV450-Registreringsapplikation/issues/1","id":58044807,"number":1,"title":"Peer Review (Kopia från mail)","user":{"login":"lr222gw","id":5381084,"avatar_url":"https://avatars.githubusercontent.com/u/5381084?v=3","gravatar_id":"","url":"https://api.github.com/users/lr222gw","html_url":"https://github.com/lr222gw","followers_url":"https://api.github.com/users/lr222gw/followers","following_url":"https://api.github.com/users/lr222gw/following{/other_user}","gists_url":"https://api.github.com/users/lr222gw/gists{/gist_id}","starred_url":"https://api.github.com/users/lr222gw/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lr222gw/subscriptions","organizations_url":"https://api.github.com/users/lr222gw/orgs","repos_url":"https://api.github.com/users/lr222gw/repos","events_url":"https://api.github.com/users/lr222gw/events{/privacy}","received_events_url":"https://api.github.com/users/lr222gw/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-02-18T09:30:12Z","updated_at":"2015-03-20T10:11:42Z","closed_at":"2015-03-20T10:11:42Z","body":"Hej! \r\nHar kollat över din kod nu, den ser bra ut! \r\n\r\n\r\nProblem:\r\nJag kom så långt att jag kunde köra \"bundle install\", men när jag försökte köra \"rails s\" så fick jag ett error om bcrypt.\r\n\r\nLöste problemet genom att byta ut föregående raden:\r\n\"gem 'bcrypt',               '3.1.7'\"\r\n\r\nmot raden:\r\n\r\ngem 'bcrypt', '~> 3.1.9' \r\n(Det var alltså fel version på bcrypt... som ej fungerade med windows..)\r\n\r\n\r\nSamma sak gällde  sqlite3, lösningen blev följande:\r\ngem 'sqlite3',     '1.3.9'\r\nbyttes ut till:\r\ngem 'sqlite3',     '1.3.10'\r\n\r\nKörde \"rake db:migrate\" och \"rake db:seed\" följt av  \"rails s\".\r\nNu fungerade sidan och allt som det skulle.\r\n\r\nÖversikt:\r\nApplikationen ser mycket bra ut! \r\nDen fungerar klockrent och hade inga problem att testa allt man kunde göra.\r\nEnda problemen jag hade var pågrund av Ruby på windows 7. \r\nKoden är enkel att läsa och skriven enligt tänket \"feta modeller, smala kontrollrar\" vilket som är toppen då vi kodar i Ruby! \r\nJag är ingen expert på RoR men jag lade inte märke till några konstigheter gällande koden eller uppdelingen av uppgifter mellan kontrollrarna och modellerna. \r\nDet verkar som du nyttjat RoR väl, då jag ser att du använt funktioner som jag itne lagt märke till,Tex \"will_paginate\" i \"index.html.erb\".\r\nTycker allt ser väldigt bra ut! \r\n\r\n\r\n\r\nHär kommer anmärkningar:\r\n-När man tar bort en applikation får man ett meddelande om att \r\napplikationen lyckades bli bortagen. Men man får också ett felmeddelande \r\nsom inte verkar relevant \"Något oväntat har gått fel, applikationen kan inte hittas i databasen\". (Gäller när man är inloggad som admin)\r\n\r\n-Inte så mycket kommenterar (dock ej så nödvändigt då koden blir kort och väldigt självförklarande...)\r\n\r\n\r\nNågra av sakerna jag testade som fungerade bra!:\r\n-Snygg.\r\n-kan inte logga in med ogiltiga uppgifter.\r\n-Kan registrera flera applikationer på en användare.\r\n-får varning i fall jag vill ta bort applikationen.\r\n-Får bra relevanta felmeddelanden.\r\n-Bra Admin gränssnitt (kan ta bort användare och deras appar enkelt).\r\n-När jag skriver fel vid registrering så finns mina uppgifter kvar.\r\n-Fungerar bra att ta lägga till applikationer.\r\n-Kommer inte in på \"http://localhost:3000/users\" med användare som ej är admin.\r\n-Använder inte get för att ta bort en användare utan delete = nice.\r\n\r\n\r\n\r\n10 av 10! "}}