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 @context when missing #54

Merged
merged 3 commits into from
May 11, 2023
Merged

Conversation

ships
Copy link
Collaborator

@ships ships commented May 11, 2023

Description

The Docmap codec will now always return a value with @context key set. If the @context key is set to something other than the correct context, it will fail to decode (do we want this behavior?) If it is empty or null, it will be set to the correct key.

Related Issues

This should improve interoperability with JSON-LD.

Checklist

  • I have tested these changes locally and they work as expected.
  • I have added or updated tests to cover any new functionality or bug fixes.
  • I have updated the documentation to reflect any changes or additions to the project.
  • I have followed the project's code of conduct and conventions for commit messages.

Additional Information

Provide any additional information that might be helpful in understanding this pull request, such as screenshots, links to relevant research, or other context.

@ships ships merged commit d003311 into main May 11, 2023
@ships ships deleted the ships/feat/add-context-key-to-docmap-result branch August 22, 2023 17:46
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.

1 participant