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

Patch permalinks #152

Merged
merged 6 commits into from
Jan 29, 2025
Merged

Patch permalinks #152

merged 6 commits into from
Jan 29, 2025

Conversation

mrtn78
Copy link
Member

@mrtn78 mrtn78 commented Nov 28, 2024

updated permalinks to correct versions
added the old versions and version 2.0.1
@mrtn78 mrtn78 requested a review from sanderke November 28, 2024 17:06
@mrtn78 mrtn78 self-assigned this Nov 28, 2024
@mrtn78 mrtn78 added documentation Improvements or additions to documentation Scope: Klein Kleine wijzigingen met beperkte scope Status: Klaar voor release Het voorstel is verwerkt en klaar voor de volgende release. Type: Correctie Correctie van een (kleine) fout. Overleg: TO API Te agenderen voor het Technisch Overleg ADR - API Design Rules labels Nov 28, 2024
@mrtn78 mrtn78 requested a review from TheBonheurs December 13, 2024 15:23
Copy link
Member

@TheBonheurs TheBonheurs left a comment

Choose a reason for hiding this comment

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

Wat vragen puur voor de verduidelijking
Verder lijken de links allemaal te werken 👍

Copy link
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.

Fixes zien er allemaal goed uit. Ik stel voor dat we dit mergen naar main, zodat we daar de permalinks "inbakken". Dus op develop gebruiken we relatieve links en dan als we de standaard indienen bij het forum pakken we de specifieke links op basis van de dan (compatibele) versies.

Dus in dit geval geven we aan dat ADR Transport Security module version 1.0.x is, dus we kijken op de dag van indienen welke laatste compatibele versie er is (dat is dus 1.0.2) en zetten die er in.

@TimvdLippe TimvdLippe requested a review from bwbroersma January 22, 2025 14:33
@mrtn78 mrtn78 mentioned this pull request Jan 23, 2025
Hiermee linken we altijd naar 3.0.1 wat consistent is met
de huidige PTOLU status bij het Forum. Tevens gebruiken
we de 3.0 notatie om specifieke referenties naar een patch
versie te voorkomen.
@TimvdLippe TimvdLippe merged commit 6c2521f into develop Jan 29, 2025
5 checks passed
@TimvdLippe TimvdLippe deleted the patch-permalinks branch January 29, 2025 09:38
TimvdLippe added a commit that referenced this pull request Feb 17, 2025
Hiermee specificeren we de exacte minor versie van de
modules die in de core zitten. Daarmee voldoen we
aan de eis dat standaarden op de PTOLU lijst enkel
vastgestelde documenten verwijzen en geen werkversies
of versies gepubliceerd in de toekomst.

Tevens refereren we naar OpenApis 3.0, dus zonder
de patch versie, om zo consistent te zijn met de huidige
staat van de PTOLU.

---------

Co-authored-by: Tim van der Lippe <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation Overleg: TO API Te agenderen voor het Technisch Overleg ADR - API Design Rules Scope: Klein Kleine wijzigingen met beperkte scope Status: Klaar voor release Het voorstel is verwerkt en klaar voor de volgende release. Type: Correctie Correctie van een (kleine) fout.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ADR 2.0.0 has non permalinks (e.g. to sub modules)
5 participants