-
Notifications
You must be signed in to change notification settings - Fork 3
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
githubjimmeicatt
wants to merge
1
commit into
main
Choose a base branch
from
pc-638-ONN
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -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)** | | ||
|
||
#### 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> | ||
|
@@ -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)** | | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe 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 | | ||
|
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
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
enINTERNE_TAAK_CLIENT_ID
)Moet dat dan niet ook worden:
niet te gebruiken met
INTERNE_TAAK_CLIENT_SECRET
enINTERNE_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?