Closed hasanheroglu closed 1 year ago
Just for further clarification:
I think it looks good. I added some comments (see below).
The main question I have is "why" we did not use node-wot for the exposed things. I see benefits of being neutral but somehow we partly implement it twice...
So the node-wot Things will also come but my idea is to not couple the protocol implementation of a Thing to the changes in node-wot. There are many changes happening to node-wot that do not need to be applied to these Things. At the same time, a longer goal is to have these as stable as possible so that node-wot consumer side can be tested against them.
So the node-wot Things will also come but my idea is to not couple the protocol implementation of a Thing to the changes in node-wot. There are many changes happening to node-wot that do not need to be applied to these Things. At the same time, a longer goal is to have these as stable as possible so that node-wot consumer side can be tested against them.
Following you arguments I would say node-wot things should than never come. Having both does not make a lot of sense.. I think
Talked with @danielpeintner as well and we can merge it. I will create a discussion for the overall management of such Things
This is the initial code for test-things. In development, I have worked with @egekorkan.
The development has started at https://github.com/thingweb/device-sandbox but that repository will be closed/archived. Already existing devices could be hosted at plugfest.thinweb.io and we want to extend the repository with more devices in the future.
For feedback on the overall structure, please provide a comment and not a GitHub review (it can be the final comment of a GitHub review).
It is a big initial contribution but we wanted to show the overall structure. The actual code is small, there is a wrapper and test code.