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.

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

Remove extra form for things propery #282

Merged
merged 2 commits into from Mar 7, 2022
Merged

Remove extra form for things propery #282

merged 2 commits into from Mar 7, 2022

Conversation

farshidtz
Copy link
Member

@farshidtz farshidtz commented Mar 7, 2022 •
edited by pr-preview bot

Closes #277

This PR:

  • removes the first form of the things property because the second from covers the same operation, given that uri variables are optional
  • adds the missing format uri variable

Preview | Diff

@farshidtz farshidtz marked this pull request as ready for review March 7, 2022 15:41
@mmccool mmccool merged commit e75026e into w3c:main Mar 7, 2022
@farshidtz farshidtz deleted the tm-things-forms branch March 7, 2022 16:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Validate things interaction forms in TM
2 participants
@farshidtz @mmccool