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

fix: add wotsec context on securityDefinitions property #1168

Merged
merged 1 commit into from
Jun 30, 2021
Merged

fix: add wotsec context on securityDefinitions property #1168

merged 1 commit into from
Jun 30, 2021

Conversation

ethieblin
Copy link
Contributor

Should fix the context to obtain the same result as the RDF example #1167

vcharpenay
vcharpenay approved these changes Jun 9, 2021
View reviewed changes
Copy link
Contributor

@vcharpenay vcharpenay left a comment

Choose a reason for hiding this comment

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

the next TD version will likely change how securityDefinitions should be processed in RDF, but this fix is satisfactory until then.

@w3cbot
Copy link

w3cbot commented Jun 30, 2021

ashimura marked as non substantive for IPR from ash-nazg.

@vcharpenay vcharpenay merged commit 241ede2 into w3c:main Jun 30, 2021
@ethieblin ethieblin deleted the fix-security-definition-ctx branch June 30, 2021 13:29
@vcharpenay vcharpenay mentioned this pull request Jun 30, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vcharpenay vcharpenay vcharpenay approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@ethieblin @w3cbot @vcharpenay