Skip to content
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

Add additional explanations to vocabulary creation guide #298

Merged
merged 10 commits into from
Dec 6, 2023

Conversation

JKRhb
Copy link
Member

@JKRhb JKRhb commented Sep 26, 2023

Based on @egekorkan's work in #282, I tried to add some initial explanations on the file structure of a binding template to the VOCABULARY-CREATION-GUIDE.md file. There is most certainly still a lot of room for improvement, but the document should now contain a baseline for further explanations of the creation process – let me know if you see something that should be added to this PR :)


Preview | Diff

@netlify
Copy link

netlify bot commented Sep 26, 2023

Deploy Preview for wot-binding-templates ready!

Name Link
🔨 Latest commit 0244830
🔍 Latest deploy log https://app.netlify.com/sites/wot-binding-templates/deploys/657098a4bc79d60008ea302c
😎 Deploy Preview https://deploy-preview-298--wot-binding-templates.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@JKRhb JKRhb requested a review from relu91 September 26, 2023 07:03
@egekorkan
Copy link
Contributor

Call of 27.09: Since @mjkoster needs this for the BACnet binding, he will review it as a user.

VOCABULARY-CREATION-GUIDE.md Outdated Show resolved Hide resolved
VOCABULARY-CREATION-GUIDE.md Outdated Show resolved Hide resolved
VOCABULARY-CREATION-GUIDE.md Outdated Show resolved Hide resolved
VOCABULARY-CREATION-GUIDE.md Outdated Show resolved Hide resolved
VOCABULARY-CREATION-GUIDE.md Outdated Show resolved Hide resolved
VOCABULARY-CREATION-GUIDE.md Outdated Show resolved Hide resolved
VOCABULARY-CREATION-GUIDE.md Outdated Show resolved Hide resolved
VOCABULARY-CREATION-GUIDE.md Outdated Show resolved Hide resolved
Copy link
Member

@relu91 relu91 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Additionally to the comments below, I think a pipeline schematic made with mermaid would help the reader to understand the flow.

VOCABULARY-CREATION-GUIDE.md Outdated Show resolved Hide resolved
VOCABULARY-CREATION-GUIDE.md Outdated Show resolved Hide resolved
VOCABULARY-CREATION-GUIDE.md Outdated Show resolved Hide resolved
VOCABULARY-CREATION-GUIDE.md Outdated Show resolved Hide resolved
VOCABULARY-CREATION-GUIDE.md Show resolved Hide resolved
VOCABULARY-CREATION-GUIDE.md Outdated Show resolved Hide resolved
VOCABULARY-CREATION-GUIDE.md Outdated Show resolved Hide resolved
VOCABULARY-CREATION-GUIDE.md Outdated Show resolved Hide resolved
JKRhb and others added 4 commits November 30, 2023 11:48
@egekorkan
Copy link
Contributor

Call of 06.12: Merging based on the review.

@egekorkan egekorkan merged commit 7e4d9c1 into w3c:main Dec 6, 2023
3 checks passed
@JKRhb JKRhb deleted the expanded-guidance branch February 13, 2024 11:44
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.

5 participants