Skip to content

Beheer: sta relatieve Server API URL toe.#291

Merged
TimvdLippe merged 3 commits into
Logius-standaarden:developfrom
kad-sprenr:patch-1
May 18, 2026
Merged

Beheer: sta relatieve Server API URL toe.#291
TimvdLippe merged 3 commits into
Logius-standaarden:developfrom
kad-sprenr:patch-1

Conversation

@kad-sprenr
Copy link
Copy Markdown
Contributor

Zie #290

@github-actions github-actions Bot added Status: In bewerking Het voorstel is in bewerking bij de beheerorganisatie. Overleg: TO-API Te agenderen voor het Technisch Overleg API labels Jan 21, 2026
Copy link
Copy Markdown
Contributor

@TimvdLippe TimvdLippe left a comment

Choose a reason for hiding this comment

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

Zoals beschreven in #290 (comment) moeten we eerst een design regel hierover definiëren (of iig de discussie of we dat moeten doen) voordat we de linter configuratie kunnen aanpassen.

Comment thread linter/spectral.yml Outdated
@kad-sprenr
Copy link
Copy Markdown
Contributor Author

@TimvdLippe Laat vooral weten hoe je verder wilt gaan met deze PR n.a.v. het TO.

@TimvdLippe
Copy link
Copy Markdown
Contributor

Gisteren heb ik #294 geupdate en daar ook een fix in meegenomen. Afhankelijk van de beslissing op het TO wordt ofwel het gehele pakket of deels gemergt. We zorgen er sowieso voor dat na het TO jouw issue wordt voorkomen, ongeacht de uiteindelijke beslissing of er wel/niet een regel voor moet komen.

@TimvdLippe TimvdLippe changed the title Sta relatieve Server API URL toe. Beheer: sta relatieve Server API URL toe. Feb 17, 2026
@github-actions github-actions Bot removed the Status: In bewerking Het voorstel is in bewerking bij de beheerorganisatie. label Feb 17, 2026
@TimvdLippe TimvdLippe removed the Overleg: TO-API Te agenderen voor het Technisch Overleg API label Feb 17, 2026
@TimvdLippe TimvdLippe dismissed their stale review May 18, 2026 13:59

De feedback is nu verwerkt

Copy link
Copy Markdown
Contributor

@TimvdLippe TimvdLippe left a comment

Choose a reason for hiding this comment

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

Er zat wat vertraging op, maar deze linter configuratie moeten we nog updaten. Thanks voor de suggestie! De design rule is niet goedgekeurd, dus we gaan geen design rule opschrijven over het wel/dan wel niet toe staan van relatieve URLs.

@TimvdLippe TimvdLippe merged commit 6edd13c into Logius-standaarden:develop May 18, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants