An Schnittstellen Definition anpassen!

[Neram]

Bitte ALLE überprüfen, ob die Parameterübergabe und Rückgabe den vereinbarten Schnittstellendefinition https://github.com/Roba1993/Happy-Hour/wiki/Frontend-zum-Backend-Schnittstellen#get entspricht und anschließend im Browser überprüfen, ob die richtigen Daten bei einer Anfrage zurückgegeben wrden! Anschließend hakt ab, ob ihr es erledigt habt!

• [x] /bars @TobiasH1993
• [ ] /routes/:hash @Tabea-Kraus
• [x] /bars/reports @stegmajo
• [ ] /routes @Micash0711
• [x] /bars/{id}/reports @stegmajo
• [ ] /report/{id} @Tabea-Kraus
• [x] /bars/{id}/reports @stegmajo
• [ ] /bars/{id}/hour @mighty190
• [x] /delHour/{hourId} @stegmajo

z.B. Tobi hat einen Integer als Radius verwendet, dies muss aber laut den vereinbarten Schnittstellendefinitionen ein Double sein. Also bitte auch so anpassen!

[markusthoemmes]

Zur Erläuterung: Der Radius kommt in Kilometern!

[stegmajo]

/bars/reports Rückgabe sieht anders aus, wurde eigentlich mit Frontend so abgesprochen: Rückgabewert

[
  {
    id: "",
    barId: "",
    description: ""
  }
]

@markusthoemmes

[stegmajo]

HTTP-Delete: /bars/{id}/reports ist nicht in der Frontend Dokumentation vorhanden

Update: Muss in eigener Dokumentation eingepflegt werden.

[markusthoemmes]

@stegmajo Ist angepasst

[samuelvogelmann]

Wie sieht der Fortschritt aus???

[markusthoemmes]

/routes entspricht NICHT wie hier markiert der Schnittstellendefinition! Es wird ein Array mit Hash geliefert anstelle eines Hashs.

[Micash0711]

Im Response scope der Rückgabe befindet sich der Hash

[markusthoemmes]

Aber als Element eines Arrays, nicht als String allein!

[Micash0711]

Die Rückgabe sieht jetzt so aus. @markusthoemmes

[markusthoemmes]

Auch das ist nicht korrekt! Es muss schon im data-Feld sein! Allerdings nicht in einem Array. Dasselbe gilt auch für /routes/:hash.

[markusthoemmes]

/bars liefert die HappyHours in einem komplett falschen Format!

1. Die HappyHour-Tage sind einzelne true/falseVariablen, es wurde sich jedoch auf das hier beschriebene Format verständigt: https://github.com/Roba1993/Happy-Hour/wiki/Frontend-Typ-HappyHour
2. Die Start- und Endzeiten der HappyHours sind nicht im HH:mm:ss Format, beziehungsweise nicht auf 24 Stunden gemünzt, was ebenso vereinbart war!

Des Weiteren gibt es Fehler im Barobjekt selbst:

• address wird mir als adress ausgegeben, was ebenfalls nicht der Schnittstelle entspricht

[TobiasH1993]

Die Schreibweise "adress" war mein Fehler ;) ist behoben.

Das Happy-Hour-Format werden wir zeitnah anpassen. Ebenso das Format der Öffnungszeiten.

[markusthoemmes]

/bars/{id}/reportsentspricht NICHT der Schnittstellendefinition. Die Daten werden in der vorliegenden Implementierung aus der URL ausgelesen und nicht wie vereinbart (und common sense) aus dem Body der Post-Nachricht! Vereinfachungen wurden bereits mit Robert besprochen, es muss nur request.body() ausgelesen werden um an die Description zu kommen.

[markusthoemmes]

/routes/:hash entspricht NICHT der Schnittstellendefinition. Es wird ein Objekt mit Zusatzinformationen zurückgegeben (in einem Array), das dann in einem weiteren data-Feld die Route enthält. Vereinbart und erwartet wird nur eine Route!

[samuelvogelmann]

An das Backend-Team! Wir hatten ausgemacht, dass die Schnittstellen bis Donnerstagabend bzw. Freitagabend angepasst sind! Wie sieht es aus? Ich sehe immernoch offene Punkte!

[samuelvogelmann]

Da die Schnittstellen teilweise immer noch nicht angepasst sind, wird die Arbeit von @Roba1993 übernommen. Siehe Issue 140