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

category.json schema should not allow additional properties #1171

Open
jonafato opened this issue Jul 28, 2024 · 0 comments · May be fixed by #1172
Open

category.json schema should not allow additional properties #1171

jonafato opened this issue Jul 28, 2024 · 0 comments · May be fixed by #1172

Comments

@jonafato
Copy link
Member

The category schema currently allows additional properties on category data, but these attributes are not used by the website (and cannot be used without first being validated by the schema). The schema should disallow this, which may also result in adding newly allowed values (with some currently existing examples being URLs and start / end dates of events).
jonafato added a commit that referenced this issue Jul 28, 2024
@jonafato
Disallow additional properties on category schema
574bf3a 
The intent of this change is to update the schema without any test
failures. This means that other changes that affect the data should
happen independently from the pull request associated with this commit,
and it should be kept up to date and merged once the schema tests pass.
To contribute to this change, open PRs that would correct any data that
does not conform to the JSON schema. Discussion regarding other schema
updates should happen in the associated issue at #1171.

Resolves #1171.
@jonafato jonafato linked a pull request Jul 28, 2024 that will close this issue
Draft
jonafato added a commit that referenced this issue Aug 3, 2024
@jonafato
Disallow additional properties on category schema
705019c 
The intent of this change is to update the schema without any test
failures. This means that other changes that affect the data should
happen independently from the pull request associated with this commit,
and it should be kept up to date and merged once the schema tests pass.
To contribute to this change, open PRs that would correct any data that
does not conform to the JSON schema. Discussion regarding other schema
updates should happen in the associated issue at #1171.

Resolves #1171.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Disallow additional properties on category schema pyvideo/data
1 participant
@jonafato