search

kadaster-labs / sensrnet-home

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

Update UserManualNL.md #275

Closed CelineJansen closed 2 years ago

marcvanandel commented 2 years ago

Goeie PR, @CelineJansen. We hadden zelf ook nog de nodige zaken die we dachten te kunnen 'oppoetsen': #276 😄

@kad-vossef , laten we eerst ons eigen PR mergen en dan kijken of deze wijzigingen van Celine nog op de juiste plek belanden, OK? Als dat goed gaat (wat ik wel verwacht), dan kan deze PR direct achter onze eigen PR aan gemerged worden, denk ik. Wat denk jij?

CelineJansen commented 2 years ago

Dankjewel @marcvanandel! @kad-vossef had me ook al via de mail een terugkoppeling gestuurd. Wordt echt handig hoe jullie het willen maken met de linkjes naar de documenten op 1 plek. Super!

kad-vossef commented 2 years ago

@marcvanandel: prima voorstel Marc!

kad-vossef commented 2 years ago

@marcvanandel: doe jij de squash and merge?

marcvanandel commented 2 years ago

@CelineJansen , ik heb onze wijzigingen in jouw branch / patch gemerged om jouw netto wijzigingen over te houden. Ik merge 'm nu, want ik denk dat het goede verbeteringen zijn. Mocht je daar niet (helemaal) mee eens zijn, dan kun je gemakkelijk een nieuwe patch + PR (Pull Request) maken, OK?

Dank voor je input!!! 🎉