richie-south / 1dv450 Goto Github PK
View Code? Open in Web Editor NEWWebbramverk
Webbramverk
Får svaret undefined method 'toilet_id'
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
När jag hämtar ut alla creators eller en enstaka visas password_digest. Det tänker jag inte borde synas.
Var tvungen att letta upp installationsinstruktionerna, till nästa gång är det kanske bättre att ha dom i en readme i rooten
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
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.
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.
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.
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
Finns ingen koll på att användaren är autentisierad i toilet controller
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
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
A declarative, efficient, and flexible JavaScript library for building user interfaces.
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
An Open Source Machine Learning Framework for Everyone
The Web framework for perfectionists with deadlines.
A PHP framework for web artisans
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
Some thing interesting about web. New door for the world.
A server is a program made to process requests and deliver data to clients.
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
Some thing interesting about visualization, use data art
Some thing interesting about game, make everyone happy.
We are working to build community through open source technology. NB: members must have two-factor auth.
Open source projects and samples from Microsoft.
Google ❤️ Open Source for everyone.
Alibaba Open Source for everyone
Data-Driven Documents codes.
China tencent open source team.