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 👍

Introduction.md Outdated Show resolved Hide resolved
SOTD.md Outdated Show resolved Hide resolved
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
Introduction.md Outdated Show resolved Hide resolved
Introduction.md Outdated Show resolved Hide resolved
@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
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