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

Context is included with .json() method call #128

Open
glenrobson opened this issue Dec 2, 2022 · 0 comments · May be fixed by #184
Open

Context is included with .json() method call #128

glenrobson opened this issue Dec 2, 2022 · 0 comments · May be fixed by #184
Assignees
Labels
enhancement New feature or request

Comments

@glenrobson
Copy link
Contributor

The context is included every time you call .json(). This makes sense when its a Manifest / Collection / AnnotationPage but it might not be necessary if you are trying to access things like provider or the metadata section as JSON.

The suggestion is to add a optional parameter excludeContext which defaults to False. This would give the option to just get the JSON without the context being added.

Alternative options are to only include the context for the .jsonld method or to only include the context if the object is a Manifest / Collection / AnnotationPage.

@glenrobson glenrobson added the enhancement New feature or request label Dec 2, 2022
@glenrobson glenrobson self-assigned this Dec 2, 2022
@digitaldogsbody digitaldogsbody linked a pull request Jun 11, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant