richie-south / 1dv450 Goto Github PK

Var tvungen att letta upp installationsinstruktionerna, till nästa gång är det kanske bättre att ha dom i en readme i rooten

Finns ingen koll på att användaren är autentisierad i toilet controller

När man loggat in så skriver du ut användarens appliaktioner, och det är ju bra DOCK så kommer det även en ruta som har vad som ser ut som ett utdrag ur databasen

Det går inte att uppdatera en position jag lagt till. Kan kanske ha att göra med att i metoden update position först kontrollerar om positionen finns, gör den det skickas svaret tillbaka att positionen redan finns, vilket den ju måste göra om man ska kunna uppdatera den.

Står inget om att jag behöver migrata databasen så jag får ett ActiveRecord::PendingMigrationError error. Dock är det ju tur att jag vet hur man gör det och löste felet

Går inte att uppdatera en tag. Får svaret att routen inte finns. Sedan är det samma sak här som i update position. Kontrollerar först om taggen finns, finns den skickas svaret tillbaka att taggen redan finns, vilket den ju måste göra om det ska gå att uppdatera

Provade att skicka in 1, men fick som svar att det inte fanns något toalett på den positionen. Kanske ytterligare en anledning att tvinga position när man skapar en toalett.

När jag hämtar ut alla creators eller en enstaka visas password_digest. Det tänker jag inte borde synas.

I kraven till api:et står: API:et ska i möjligaste mån grunda sig på HATEOAS. Jag kan inte se det implementerat i något JSON response. T ex skulle det varit väldigt hjälpfullt om
get positions och get position/:id hade en href till toaletten och inte bara id
get tags och get tags/id hade en href till creator

API.et skulle vara lättare att förstå och använda om JSON response innehöll mer information.
Tex:
get toilets och get toilets/:id borde ha information om toalettens position och taggar
get tags och get tags/:id borde ha info om alla toaletter som har den taggen
get position och get position/:id har ingen information om toalett som finns där
get creators get creator/:id borde ha information om vilka toaletter de skapat

Om du utökar informationen skulle vissa metoder bli överflödiga som:
get positions/toilet_id borde inte behövas om man istället fick med position i svaret när man efterfrågar en specifik toalett likadant för get tags/toilet_id

Får svaret undefined method 'toilet_id'

Det kanske finns en anledning till att man inte måste ange position när man skapar en toalett, men jag funderar på om det inte skulle vara enklare att göra det. På så sätt behöver man inte först använda en url för att skapa toaletten och en för positionen och sedan koppla dem samman. Det säkerställer också att varje toalett faktiskt får har position. Görs det på det här sättet skulle metoden create position bli överflödig.

Finns ett stavfel på rad 10 i indexmetoden som ställer till det.
Samma sak gäller creators/:id stavfel på respond_with. Här är det även fel på rad 16 där du använder varabelnamnet c istället för creator