Grenmyr
commented
9 years ago jag har tittat på din feedback och kommenterat till din feedback.
https://github.com/Grenmyr/1DV450Registration/issues/7
Jag tolkar dig som det e inget problem med gokänt, men att mitt tankemönster + hateoas e lite otydlig. Jag editerade din comment med mina svar (kanske dumt [?]) Men Jag hoppas det kanske förklarar hur jag tänkt, för det känns som en del saker du kommenterade på (vilket jag uppskattar) var missuppfattning. Antagligen pga dålig instruktioner eller kommentering i kod av mig.
Jag kommer nog uppdatera mitt API ju mer jag jobbar med angular appen, kanske lägga till columner eller routes vid behov. Men som jag tolkar dig så är Laboration 2 rails ändå godkänd?
Den 9 mars 2015 16:05 skrev John H notifications@github.com:
Jag har kollat igenom ditt API lite genom att testköra dina POSTmanfrågor. PÅ det hela taget är själva kodandet helt ok jag är kasnke(?) lite konfunderad över hur hela din tjänst är tänkt att fungera. Antar det komemr framgå när du skriver klienten. Jag tror det är lite förvirrande med namnet "event".
När man gör en inloggning kör du en GET och skicka rmed användarnamn och lösenord som header-field. Det normala är kanske att köra en POST och skicka med dessa som parametrar vilket brukar vara det normala fallet vid t.ex. formulär
Som jag förväntade mig APIet så skapar en användare ett event (denne har hittat ett svampställe), ger det en rubrik samt en position. Men när jag kör events/1 så har den massa positioner samt endast en skapare. Då antar jag att APiet håller mer reda på en specifiks användares alla svampställen?
Jag kan också tycka du är lite snål i ditt API på vissa ställen. T.ex. ber jag få en användare (creator) så får man en siffra på antal event (amount)...men jag skulle ju vilja ha information eller länkar till dessa event så min klientapplikation kan jobba vidare med dessa. Likadant vid positions/1 där jag inte får fram någon mer information än om long, lat. Det kommer göra APIet granska pratigt och klienten behöver göra många anrop. Det saknas lite nästlad information samt en tanke kring HATEOAS. Kanske du kommer märka i klientapplikationen.
T.ex: kan man på något ställe göra en sökning på en plats och få ut de "event/händelser" som är knutna till den platsen eller missar jag den delen? Kan man skriva en sökfråga som söker händelse på lämpligt sätt
Kanske missar jag något i dina postman-filer...
— Reply to this email directly or view it on GitHub https://github.com/Grenmyr/1DV450Registration/issues/7.
1dv450
Jag har kollat igenom ditt API lite genom att testköra dina POSTmanfrågor. PÅ det hela taget är själva kodandet helt ok jag är kasnke(?) lite konfunderad över hur hela din tjänst är tänkt att fungera. Antar det kommer framgå när du skriver klienten. Jag tror det är lite förvirrande med namnet "event".
Ja bättre namn kanske skulle varit findings eller något sånt.
När man gör en inloggning kör du en GET och skicka rmed användarnamn och lösenord som header-field. Det normala är kanske att köra en POST och skicka med dessa som parametrar vilket brukar vara det normala fallet vid t.ex. formulär
Tar till mig det, tycker ändå det inte påverkar säkerhetsmässigt, men man ska ju följa standard.
Jag kan också tycka du är lite snål i ditt API på vissa ställen. T.ex. ber jag få en användare (creator) så får man en siffra på antal event (amount)...men jag skulle ju vilja ha information eller länkar till dessa event så min klientapplikation kan jobba vidare med dessa. Likadant vid positions/1 där jag inte får fram någon mer information än om long, lat. Det kommer göra APIet granska pratigt och klienten behöver göra många anrop. Det saknas lite nästlad information samt en tanke kring HATEOAS.
Ok, ja vill du ha länkarna får man skriva creator/:id/events. Kanske skulle haft mer Hatoas.
T.ex: kan man på något ställe göra en sökning på en plats och få ut de "event/händelser" som är knutna till den platsen eller missar jag den delen? Kan man skriva en sökfråga som söker händelse på lämpligt sätt
Ja, du har två parametrar, location och range. I range skriver du sökrange. Och i location skriver du antingen [lat,lng] om du vill ha på lat lnt, eller kalmar om du vill söka från stad.
Kanske missar jag något i dina postman-filer...För mig blev det lite rörigt med många liknande anrop