{"action":"closed","issue":{"url":"https://api.github.com/repos/ch222kv/ch222kv_1dv450_kod/issues/9","labels_url":"https://api.github.com/repos/ch222kv/ch222kv_1dv450_kod/issues/9/labels{/name}","comments_url":"https://api.github.com/repos/ch222kv/ch222kv_1dv450_kod/issues/9/comments","events_url":"https://api.github.com/repos/ch222kv/ch222kv_1dv450_kod/issues/9/events","html_url":"https://github.com/ch222kv/ch222kv_1dv450_kod/issues/9","id":60827584,"number":9,"title":"Kommentarer API","user":{"login":"1dv450","id":6397336,"avatar_url":"https://avatars.githubusercontent.com/u/6397336?v=3","gravatar_id":"","url":"https://api.github.com/users/1dv450","html_url":"https://github.com/1dv450","followers_url":"https://api.github.com/users/1dv450/followers","following_url":"https://api.github.com/users/1dv450/following{/other_user}","gists_url":"https://api.github.com/users/1dv450/gists{/gist_id}","starred_url":"https://api.github.com/users/1dv450/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/1dv450/subscriptions","organizations_url":"https://api.github.com/users/1dv450/orgs","repos_url":"https://api.github.com/users/1dv450/repos","events_url":"https://api.github.com/users/1dv450/events{/privacy}","received_events_url":"https://api.github.com/users/1dv450/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2015-03-12T14:32:06Z","updated_at":"2015-03-22T22:20:25Z","closed_at":"2015-03-22T22:20:25Z","body":"Hejsan!\r\nSent om sider kommer lite små kommentarer kring ditt API.\r\nKodmässigt ser det bra ut och man hittar små eleganta lösningar här och var. Kul du testar på postgreSQL och deras lösning för sökning. Dock tycker jag som din peer att du bör ge lite klarare instruktioner kring installation (länk till nedladdning) och kommandon för skapandet där. Kan tänka det tar lite tid för en ovan att lista ut dessa saker.\r\n\r\nJag saknar också en mer komplett seedfil där  du lägger in event, api-nyklar och annat som behövs för att testa ditt API snabbt och enkelt. Det ska man inte behöva skapa själv. Skapa in en sådan till slutliga redovisningen.\r\n\r\nJag testade det jag kunde och responsen ser bra ut. Känns som vettiga svar men det är lite svårtestat när man inte har någon vettig data att testa på. Jag får endel fel på förfrågningar mot specifika resurser, sökningar m.m. där man får en röd felsida och inte något felmeddelande i json. Förmodligen beror det på att databasen är tom?\r\n\r\nJag är inte helt med på hur din säkerhet och au som slutanvändare fungerar. När man registrerar i din applikation får man ett id och en secret. Antar client är API-nyckeln för den funkade på de flesta anrop. Kanske kan klargöra lite för hur det fungerar.\r\n\r\nHur autentiserar man sig mot API:et som slutanvändare för att kunna skapa, ändra och ta bort resurser? Jag hittar inget sådant i koden och bara GET-förfrågningar i din postman? Det känns som \"API-nycklen\" används som Autentisering av slutanvändarna? Den borde väl användas för att identifiera applikationen och inte slutanvändaren som borde få en egen token vid inloggningen via API:et? Att man skiljer på resursägare (slutanvändarna) och applikationsutvecklarna (de som registrerar sig och får en \"client_key\").\r\n\r\nSom sagt det är bra kod och snygga lösningar men lite mer information och hjälp (seed) kan behövas för att testa din applikation på ett bättre sätt då jag inte vill sitta och skriva in massa data själv.\r\n\r\n"}}