search

kadaster-labs / sensrnet-home

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

Kad bloemy patch 1 #292

Closed kad-bloemy closed 2 years ago

kad-bloemy commented 2 years ago

Ik heb eerst Definitions.rst aangepast, maar omdat ik lekker bezig was, heb ik ook een Definitions.md gemaakt. Is mooier. Tevens alle verwijzingen naar .rst gewijzigd naar .md.

kad-busses commented 2 years ago

Goeie @marcvanandel, dat had ik niet gecheckt, jammer dat die rendering niet goed werkt. Zouden we, in plaats van alles weer in Excel te stoppen, niet simpelweg een linkje toevoegen naar de GitHub pagina voor de definities?

marcvanandel commented 2 years ago

Yup. Dat kan zeker! En zo was het ook (alleen was het reStructuredText ipv Markdown ;-)

kad-bloemy commented 2 years ago

Dank voor het commentaar. Voorgestelde aanpassingen van Stefan aangebracht. De link naar het E-mail mask was al verbeterd, maar kennelijk niet in jullie versie. Moet ik nu een nieuw pull-request maken?

marcvanandel commented 2 years ago

Nee hoor, dit pull request kun je gewoon bijwerken ...

marcvanandel commented 2 years ago

Als we Definitions.rst vervangen door iets anders (bijv. een verbeterde Definitions.md) dan moeten we niet vergeten om de .rst wel te verwijderen ... (en links daar naartoe ook aan te passen)

kad-bloemy commented 2 years ago

@marcvanandel Ik had in mijn branch volgens mij al alle linkjes aangepast naar .md, behalve die in de documentatie publicatie.

marcvanandel commented 2 years ago

Ik denk dat het zo wel goed (genoeg) is ... Zullen we 'm mergen (incl. squash!! 😉 )