Skip to content

Hard coded highlightHref - #109

Closed
sanderke wants to merge 1 commit into
developfrom
highlight-hardcode
Closed

sanderke wants to merge 1 commit into
developfrom
highlight-hardcode

Conversation

@sanderke

Copy link
Copy Markdown
Member

Met succes getest, maar het is een lompe wijziging in core dat eigenlijk in nlgov behoort.

@sanderke
sanderke requested a review from TimvdLippe August 26, 2026 09:25

@TimvdLippe TimvdLippe left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Ik snap niet hoe dit het issue fixt. We laden de respec build in vanuit https://logius-standaarden.github.io/publicatie/respec/organisation-config.mjs dus waarom zou deze import dan kaduuk gaan?

Als het het lokaal fixt, dan tja so-be-it. Maar we hardcoden hier wel naar de GitHub versie en dat lijkt mij niet wenselijk.

Kunnen we niet beter de respec-highlight weer verwijderen? Dan zou het ook niet moeten falen.

@sanderke

Copy link
Copy Markdown
Member Author

Het succes bleek te danken aan een lokale versiewijziging.

@TimvdLippe

Copy link
Copy Markdown

Ik zie dat in Logius-standaarden/OAuth-NL-profiel#140 de URL toch localhost was. Dat is inderdaad niet de goeie.

Als dat het geval is, dan ben ik ook oke met linken naar onze versie. Maar misschien dan ipv github.io een link naar gitdocumentatie?

@sanderke

Copy link
Copy Markdown
Member Author

Misschien toch twee losse oplossingen voor het probleem. De build uit deze PR lijkt te werken in Github zonder verdere wijzigingen:

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.

2 participants