Logius-standaarden / respec

A tool for creating technical documents and web standards
https://respec.org/
Other
1 stars 4 forks source link

localBiblio kunnen opnemen in zowel organisation-config.js als config.js #52

Open lvdbrink opened 1 year ago

lvdbrink commented 1 year ago

Is your feature request related to a problem? Please describe. Het zou mooi zijn als je een localBiblio kunt opnemen in organisation-config.js én eentje in config.js. Dan kun je organisation-config.js ergens op een centrale plek zetten, met daarin de parameters die voor alle documenten in je organisatie gelden, én met alle gangbare standaarden waarnaar verwezen wordt. Een individueel respec document kan dan via config.js nog extra standaarden/documenten opnemen in de biblio, die minder gangbaar zijn maar wel relevant voor dat document.

[edit]Momenteel kun je localBiblio niet in beide config bestanden opnemen. De localBiblio in organisation-config wordt dan volledig overschreven en entries die daarin staan worden niet meer gevonden. [/edit]

Describe the solution you'd like localBiblio kunnen opnemen in zowel organisation-config.js als config.js. De laatste vult de eerste aan en overschrijft entries met dezelfde name.

Describe any alternatives you've considered Andere optie zou zijn om te zorgen dat de gangbare standaarden waarnaar we vaak verwijzen allemaal in specref staan.

wilkoquak commented 11 months ago

Deze staat al 6 maanden open: ik kan me wel goed voorstellen dat er 3 bibliografie"en zijn dat er eerst in de biblio voor de publicatie gezocht wordt, daarna in de organisatie en als laatste specref.

sanderke commented 11 months ago

Bedankt, deze was inderdaad uit het zicht verdwenen.

Wat ReSpec betreft is er maar één configuratieobject. De twee objecten worden samengevoegd in index.html:

<script class="remove"> respecConfig = {...organisationConfig, ...respecConfig}</script>

Indien er overlap is zal deze door de laatste worden overschreven – de configuratie in het document (respecConfig) in dit geval. De wens hier is om localBiblio niet te overschrijven, maar juist samen te voegen. Dit is te doen door deze regel toe te voegen vóór de eerder genoemde samenvoeging:

respecConfig.localBiblio = Object.assign(organisationConfig.localBiblio,respecConfig.localBiblio);

Voorgestelde wijziging in index.html:

-  <script class="remove"> respecConfig = {...organisationConfig, ...respecConfig}</script>
+  <script class="remove">
+    respecConfig.localBiblio = Object.assign(organisationConfig.localBiblio,respecConfig.localBiblio);
+    respecConfig = {...organisationConfig, ...respecConfig};
+  </script>