search

Nasdaq / data-link-python

A Python library for Nasdaq Data Link's RESTful API
MIT License
428 stars 70 forks source link

jsondate is broken, abandoned and unmaintained #24

Open yurivict opened 2 years ago

yurivict commented 2 years ago

Please remove it in favor of some other way to handle json.

Project was last updated 10 years ago, and their example is broken.

couture-ql commented 2 years ago

Interesting. We're only using jsondate for the test suite. The majority of calls end up putting our response payloads into a pandas data frame. I wonder if the tests should be updated to use some other internals from some other wrapper to deserialize.

As it's not breaking our tests yet, I can't imagine we'll get to this very soon. Unless you have a recommendation, I'll need to kick this down the road for a bit when I can get someone on the team to investigate a saner wrapper. It's likely a simple fix, but there are some other priorities we're looking into.