Skip to content

Architecture Assertions from Ege #2171

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.

Sign up for GitHub

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

Jump to bottom
Merged
egekorkan merged 3 commits into from
Dec 18, 2025
Merged

Architecture Assertions from Ege #2171

egekorkan merged 3 commits into from
Dec 18, 2025

Conversation

@egekorkan
Copy link
Contributor

@egekorkan egekorkan commented Dec 10, 2025 •
edited by pr-preview bot
Loading

Description of Changes

I am adding the assertions under my name. The formulations are not perfect yet, they don't fit the surrounding text very well. To be worked on.

Related Issue

#2151

Type of Change

Preview | Diff

@egekorkan egekorkan changed the title docsync: add new arch assertions Architecture Assertions from Ege Dec 10, 2025
@egekorkan egekorkan marked this pull request as ready for review December 11, 2025 11:45
@egekorkan egekorkan requested a review from k-toumura December 11, 2025 11:45
@egekorkan egekorkan added normative change document synchronisation topics on how to better synchronize between other WoT documents labels Dec 11, 2025
@k-toumura
Copy link
Contributor

k-toumura commented Dec 11, 2025

index.html file adds three assertions (24,16,17), but index.template.html only has one (17).
Is this intended?

@egekorkan
Copy link
Contributor Author

egekorkan commented Dec 11, 2025

TD call:

  • We are leaving out 20. Interaction Affordances MUST include one or more Protocol Bindings. ([arch-hypermedia](https://www.w3.org/TR/wot-architecture11/#arch-hypermedia)). In the introduction (informative) this is mentioned and then the Form at https://w3c.github.io/wot-thing-description/#interactionaffordance also mentions this
  • index.html file adds three assertions (24,16,17), but index.template.html only has one (17). Is this intended? -> The other two are added in the ttl file.

danielpeintner
danielpeintner reviewed Dec 11, 2025
View reviewed changes
@egekorkan
Copy link
Contributor Author

egekorkan commented Dec 18, 2025

TD Call today: Merging

@egekorkan egekorkan merged commit 782c750 into main Dec 18, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@danielpeintner danielpeintner danielpeintner left review comments

@k-toumura k-toumura k-toumura approved these changes

Assignees

No one assigned

Labels

document synchronisation topics on how to better synchronize between other WoT documents normative change

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@egekorkan @k-toumura @danielpeintner