search

kadaster-labs / sensrnet-home

Home of the SensRNet - The Dutch National Sensor Registry Network
Other
15 stars 7 forks source link

Update README.md #294

Closed kad-bloemy closed 2 years ago

kad-bloemy commented 2 years ago

Link naar definitions.md op Github

kad-bloemy commented 2 years ago

Ik meende een andere README te hebben aangepast ( die op https://kadaster-labs.github.io/sensrnet-home/), maar dat lijkt toch dezelfde te zijn als die in Github, die ik al heb aangepast. In dat geval zou deze weg kunnen

marcvanandel commented 2 years ago

Deze wijziging kan in het andere PR gemaakt worden ... waar je dat inderdaad al had gedaan ... maar wel met een andere link (relatief en niet de hele lange en absolute URL). Het moet met de lange, absolute URL zijn ... in de andere PR 😉

kad-bloemy commented 2 years ago

Dus dit PR kan weg. De andere PR bevat de relatieve link en hoewel ik niet helemaal begrijp wat je bedoelt (lange in de andere PR?, relatief in de andere PR?) denk ik dat ie daar goed staat.🤔 Excuus voor mijn nitwit drama's.

marcvanandel commented 2 years ago

No worries 😃

hier (in de files van deze PR) staat wat je gewijzigd hebt. Tussen haakjes staat waar de link naar verwijst. Dat is https://github.com/kadaster-labs/sensrnet-home/blob/main/docs/Definitions.md, een 'absolute' URL en ... nogal lang 😆

In de andere PR (#292) heb je ook de README.md aangepast ... maar daar die link verwijst naar 'alleen' Definitions.md. Dat is (dus) een relatieve link tov de pagina waar je op dat moment bent (deze README.md). Maar ... omdat de definitions zo'n grote tabel bevat en die niet goed wordt weergegeven in de documentatie 'engine' (de manier waarop we de documentatie publiceren), heb je (dus) een link nodig naar zoals deze in GitHub te vinden is: de absolute (en lange) URL. Deze dus: https://github.com/kadaster-labs/sensrnet-home/blob/main/docs/Definitions.md

(LET OP: Deze URL bestaat nog niet omdat deze nog in je (andere) PR zit ... en nog 'gemerged' moet worden naar main ... waar de link al naar verwijst 😉)

marcvanandel commented 2 years ago

Closing this PR and updating the other 😉