Skip to content

API update #113

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 13 commits into from
Jun 8, 2018
Merged

API update #113

merged 13 commits into from
Jun 8, 2018

Conversation

zolkis
Copy link
Contributor

@zolkis zolkis commented Apr 30, 2018

Work in progress.
A series of commits for changing the API as agreed in the Prague F2F, and the following issues:
#96 #108 #109 #110 #111 #112 #114 #115
Signed-off-by: Zoltan Kis [email protected]

Signed-off-by: Zoltan Kis <[email protected]>
@mkovatsc
Copy link
Contributor

mkovatsc commented May 8, 2018

zolkis added 2 commits May 14, 2018 11:16
Signed-off-by: Zoltan Kis <[email protected]>
…ns to ThingTemplate, add support for referencing type information. Update examples. Small changes on structures.

Signed-off-by: Zoltan Kis <[email protected]>
@zolkis
Copy link
Contributor Author

zolkis commented May 17, 2018

@danielpeintner please check this commit...

@zolkis zolkis force-pushed the master branch 2 times, most recently from 60f56c1 to 0771fad Compare May 23, 2018 12:04
zolkis added 2 commits May 28, 2018 16:52
…er for discovery. Improve Security dictionary. Editorial fixes.

Signed-off-by: Zoltan Kis <[email protected]>
@zolkis
Copy link
Contributor Author

zolkis commented May 28, 2018

Note that there are a few small differences between the TD and Scripting, especially in DataSchema vs ValueType. For instance in Scripting, object properties contain the information whether they are required or not (by default not required), whereas in the TD there is a separate property required as a list of strings. Traditionally each is correct in their own context (scripting dictionaries vs JSON-LD). However, this is an easy and exact mapping for the implementations.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants