Klantinteractie-Servicesysteem / KISS-frontend

Repository for the KISS frontend developed with ICATT for Dimpact
Other
0 stars 5 forks source link

Redoc Sluitend maken door vernieuwde PUB documentatie op te nemen in Algemene Redoc (refactor) #444

Closed rubenvdlinde closed 1 year ago

rubenvdlinde commented 1 year ago
WilcoLouwerse commented 1 year ago

Redoc skills en types lijsten + pub object oude velden weghalen

sytskevanhasselt commented 1 year ago

Op dit moment zie ik op de redoc alléén calls staan naar: /api/kiss_openpub_pub/{id}

In de frontend zie ik ook nog twee calls, om de types en de skills op te halen.

Deze kan ik niet vinden op de redoc.

lencodes commented 1 year ago

Mijn bevindingen ter documentatie en (jammer genoeg) nog zonder conclusie:

De huidige implementatie:

  1. We halen alle openpub_skills op via een Wordpress proxy, hier krijgen we (o.a.) het door Wordpress gealloceerd id mee per skill;
  2. We doen een GET op alle openpub_pub berichten via een Wordpress proxy en gebruiken hier als filter de id uit stap 1.

Verwachtte implementatie (ook kloppend met de huidige ReDoc):

  1. We halen alle openpub_skills op via de referentie lijst (en dus niet via Wordpress), hier krijgen we (o.a.) het door de Gateway gealloceerd id mee per skill;
  2. We halen alle berichten op via de Gateway met de id uit stap 1.

Het is mij (nog) niet duidelijk of de laatste stap mogelijk is. Mogelijke oplossingsrichtingen voor dit issue:

  1. (Waarschijnlijk het netste, maar ook het meeste werk): de Verwachtte implementatie implementeren;
  2. (Waarschijnlijk, by far, het snelste): de twee proxy calls toevoegen aan de ReDoc en de rest van de implementatie laten as-is.
sytskevanhasselt commented 1 year ago

Oplossingsrichting 1 doen we pas, als er een besluit is op #440

Besproken: Van e mogelijke oplssingen, pakken we optie 2. Met kleine aanpassing: we vermelden de twee calls naar Types en Skills in de documentatie van Openpub. Dat heeft lencodes gedaan.

sytskevanhasselt commented 1 year ago

PUB-issue, gesloten ivm. 1C