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

Fix "ImportError" for jsonschema #479

Merged
merged 1 commit into from
Sep 26, 2023

Conversation

OrangeTux
Copy link
Collaborator

This library relied on a private API of jsonschema to map a validation error to the right OCPPError. As of jsonschema==4.19.0, the private API has changed.1. Using the latest release of jsonschema with this library caused an ImportError.

This commit removes the dependency on the private API.

Fixes #471

Footnotes

  1. https://github.com/python-jsonschema/jsonschema/commit/931323043b4409a0416f373403bfb52408fc282f

@OrangeTux OrangeTux force-pushed the 471-fix-import-error-for-more-recent-json-schemas branch from a57e415 to c67ca12 Compare September 26, 2023 07:09
This library relied on a private API of jsonschema to map a validation
error to the right `OCPPError`. As of jsonschema==4.19.0, the private
API has changed.[^1]. Using the latest release of jsonschema with this
library caused an `ImportError`.

This commit removes the dependency on the private API.

[^1]: python-jsonschema/jsonschema@9313230
@OrangeTux OrangeTux force-pushed the 471-fix-import-error-for-more-recent-json-schemas branch from c67ca12 to da4eb20 Compare September 26, 2023 07:10
@OrangeTux OrangeTux changed the title Stop using private API jsonschema Fix "ImportError" for jsonschema Sep 26, 2023
@OrangeTux OrangeTux merged commit 54234d6 into master Sep 26, 2023
6 checks passed
@jainmohit2001 jainmohit2001 deleted the 471-fix-import-error-for-more-recent-json-schemas branch October 1, 2024 19:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

ImportError with jsonschema==v4.19.1
2 participants