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

Simplejson is required, but not listed in requirements. #43

Closed
pgoy-dod opened this issue Nov 3, 2021 · 1 comment · Fixed by #44
Closed

Simplejson is required, but not listed in requirements. #43

pgoy-dod opened this issue Nov 3, 2021 · 1 comment · Fixed by #44

Comments

@pgoy-dod
Copy link

pgoy-dod commented Nov 3, 2021

I do not have simplejson installed in my project. When i do ./manage.py makemigrations, i get this error message:

  File ".../urls.py", line 10, in <module>
    from .schema import schema
  File ".../schema.py", line 14, in <module>
    from reportal.diagnostics.schema import schema as diagnostics_schema
  File ".../diagnostics/schema.py", line 10, in <module>
    from .graphql.bindings import bindables
  File ".../diagnostics/graphql/bindings.py", line 9, in <module>
    from ariadne_django.scalars import datetime_scalar
  File ".../env/lib/python3.10/site-packages/ariadne_django/scalars/__init__.py", line 6, in <module>
    from .json import json_scalar, parse_json_value, serialize_json
  File ".../env/lib/python3.10/site-packages/ariadne_django/scalars/json.py", line 5, in <module>
    import simplejson as json  # supports decimals, etc.
ModuleNotFoundError: No module named 'simplejson'

Although it seems like simplejson shouldn't even be a requirement, because json in the standard library is simplejson, isn't it?

json (originally called simplejson) was written by Bob Ippolito.

@pgoy-dod
Copy link
Author

pgoy-dod commented Nov 3, 2021

I also found this StackOverflow answer which suggests using json as a fallback if simplejson isn't available:

https://stackoverflow.com/a/712799

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 a pull request may close this issue.

1 participant