Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

pc-638-ONN release maken en release configuratie aanpassen #86

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions docs/INSTALLATION.md
Original file line number Diff line number Diff line change
Expand Up @@ -176,8 +176,8 @@ Daarnaast zijn er bronnen die binnen KISS doorzocht moeten worden.
| `INTERNE_TAAK_OBJECT_TYPE_URL` | URL van het Objecttype Interne Taak <details> <summary>Meer informatie </summary> Bijvoorbeeld `https://objecttypenregister.mijngemeente.nl/api/v2/objecttypes/f83fdc48-5ddb-4b1a-a347-e20092031399` </details> |
| `INTERNE_TAAK_TYPE_VERSION` | Versienummer van het Objecttype Interne Taak <details> <summary>Meer informatie </summary> Bijvoorbeeld `2` <br /> KISS schrijft InterneTaken in het Objectenregister. Hierbij moet je altijd de versie van het objecttype meegeven. Omdat het per gemeente kan verschillen welke versie de meest recente is, moet je hier invullen welk versienummer KISS moet meegeven. </details> |
| `INTERNE_TAAK_TOKEN` | Token voor de Objecten API voor Interne Taken <br/> **(niet te gebruiken met `INTERNE_TAAK_CLIENT_SECRET` en `INTERNE_TAAK_CLIENT_ID`)** <details> <summary>Meer informatie </summary>In de meeste gevallen identificeert KISS zich bij een Objectenregistratie m.b.v. een `TOKEN`. In sommige gevallen is het nodig om de authenticatie in de Objecten API, voor Afdelingen, Groepen en Interne Taken, Medewerkers in te regelen m.b.v. een `client secret` en een `client id`. Dit is bv. het geval als je KISS gebruikt i.c.m. de e-Suite. Afhankelijk van de situatie moet je dus een Token inregelen, en in andere gevallen een id+secret. **NOOIT ALLEBEI!** </details> |
| `INTERNE_TAAK_CLIENT_SECRET` | Client Secret voor de Interne Taken API <br/> **(niet te gebruiken in combinatie met `INTERNE_TAAK_TOKEN`)** |
| `INTERNE_TAAK_CLIENT_ID` | Client ID voor de Interne Taken API <br/> **(niet te gebruiken in combinatie met `INTERNE_TAAK_TOKEN`)** |
| `INTERNE_TAAK_CLIENT_SECRET` | Client Secret voor de Interne Taken API <br/> **(niet te gebruiken in combinatie met `INTERNE_TAAK_TOKEN`, dan deze leeg laten)** |
| `INTERNE_TAAK_CLIENT_ID` | Client ID voor de Interne Taken API <br/> **(niet te gebruiken in combinatie met `INTERNE_TAAK_TOKEN`, dan deze leeg laten)** |

Copy link
Contributor

@sytskevanhasselt sytskevanhasselt Jan 6, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Twee vragen:

Je hebt hier bij Client secret en client ID expliciet toegevoegd: 'dan deze leeg laten'.
Moet er bij INTERNE_TAAK_TOKEN niet iets vergelijkbaars staan?
Daar staat nu
(niet te gebruiken met INTERNE_TAAK_CLIENT_SECRET en INTERNE_TAAK_CLIENT_ID)

Moet dat dan niet ook worden:
niet te gebruiken met INTERNE_TAAK_CLIENT_SECRET en INTERNE_TAAK_CLIENT_ID, dan deze twee leeglaten)

Daarnaast: waarom heb je dit hierbij gezet?
want ik realiseer mij dat ik iet weet wat hier precies bedoeld wordt.

Is het wellicht beter om een ""details"-sectie toe te voegen, zoals dnu ook is bij interne Taak Token?

#### KISS-Elastic-Sync
<details> <summary>Meer informatie </summary>KISS-Elastic-Sync is het component dat zorgt dat de gekoppelde bronnen die via Elasticsearch ontsloten worden in KISS, naar de juiste Indexen worden gepushed, met de benodigde gegevens hieraan toegevoegd. Onderstaande environment variabelen gaan over de bronnen die gekoppeld zijn aan de KISS-Elastic-Sync. </details>
Expand All @@ -189,8 +189,8 @@ Daarnaast zijn er bronnen die binnen KISS doorzocht moeten worden.
| `ENTERPRISE_SEARCH_PRIVATE_API_KEY` | Private API key voor Elastic |
| `MEDEWERKER_OBJECTEN_BASE_URL` | URL van de Objecten API voor medewerkers <details> <summary>Meer informatie </summary> Bijvoorbeeld: `https://objectenregister.mijngemeente.nl` </details> |
| `MEDEWERKER_OBJECTEN_TOKEN` | Token voor de Objecten API voor medewerkers <br/> **(niet te gebruiken i.c.m. MEDEWERKER_OBJECTEN_CLIENT_SECRET en MEDEWERKER_OBJECTEN_CLIENT_ID)** <details> <summary>Meer informatie </summary>In de meeste gevallen identificeert KISS zich bij een Objectenregistratie m.b.v. een `TOKEN`. In sommige gevallen is het nodig om de authenticatie in de Objecten API, voor Afdelingen, Groepen en Interne Taken, Medewerkers in te regelen m.b.v. een `client secret` en een `client id`. Dit is bv. het geval als je KISS gebruikt i.c.m. de e-Suite. Afhankelijk van de situatie moet je dus een Token inregelen, en in andere gevallen een id+secret. **NOOIT ALLEBEI!** </details> |
| `MEDEWERKER_OBJECTEN_CLIENT_ID` | Client ID voor de Objecten API voor medewerkers <br/> **(niet te gebruiken i.c.m. MEDEWERKER_OBJECTEN_TOKEN)** |
| `MEDEWERKER_OBJECTEN_CLIENT_SECRET` | Client Secret voor de Objecten API voor medewerkers <br/> **(niet te gebruiken i.c.m. MEDEWERKER_OBJECTEN_TOKEN)** |
| `MEDEWERKER_OBJECTEN_CLIENT_ID` | Client ID voor de Objecten API voor medewerkers <br/> **(niet te gebruiken i.c.m. MEDEWERKER_OBJECTEN_TOKEN, dan deze leeg laten)** |
| `MEDEWERKER_OBJECTEN_CLIENT_SECRET` | Client Secret voor de Objecten API voor medewerkers <br/> **(niet te gebruiken i.c.m. MEDEWERKER_OBJECTEN_TOKEN, dan deze leeg laten)** |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

En natuurlijk hier hetzelfde als bij Interne Taak

| `MEDEWERKER_OBJECT_TYPE_URL` | URL van het Objecttype Medewerker <details> <summary>Meer informatie </summary> Bijvoorbeeld `https://objecttypenregister.mijngemeente.nl/api/v2/objecttypes/f83fdc48-5ddb-4b1a-a347-e20092031399` </details> |
| `VAC_OBJECTEN_BASE_URL` | URL van de Objecten API voor VAC's <details> <summary>Meer informatie </summary> Bijvoorbeeld: `https://objectenregister.mijngemeente.nl` </details> |
| `VAC_OBJECTEN_TOKEN` | Token voor de Objecten API voor VAC's |
Expand Down